Conversation
5b579b3 to
5c55c7d
Compare
52dee74 to
f53bda0
Compare
Janik-Haag
left a comment
There was a problem hiding this comment.
Mostly looks good, just some small change suggestions. Also how would I test if the package is actually working, I can off-course build it (and it builds fine) but is there some simple way to test functionality?
37e90df to
3b1a6b0
Compare
|
just waiting for ci now. |
Right, I never responded to this, sorry. You can run the demo compositors in bin from the TTY (it doesn't yet support windowed mode, right @ehopperdietzel?) |
|
I might be missing something but I don't see a binary: 19 directories, 146 files |
|
Hmm, is there no 'out' output? I'll have to check in a bit, that's wrong. |
|
Yeah, it's there for me. ctrl+shift+esc to quit. |
|
Hm it's not there if I do |
|
Yes, you need to run them from the TTY, it doesn't support nested compositor yet. |
Yes, currently, it only has the DRM graphic backend, so, as you mentioned, it should be launched from a free session (tty). If you still face issues running it, setting LOUVRE_DEBUG=4 should help you understand what is happening. |
Janik-Haag
left a comment
There was a problem hiding this comment.
Okay, sorry for the trouble, one last thing you should add meta.mainProgram so nix run adn lib.getExe know what the default binary is.
3b1a6b0 to
92f5a24
Compare
|
I'll just pick louvre-views for that since that's the one with the most lines of code :P |
92f5a24 to
a5bd05a
Compare
|
@ofborg build louvre |

Description of changes
Adds the Louvre library for building Wayland compositors. It comes with a few demo compositors.
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Priorities
Add a 👍 reaction to pull requests you find important.