New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wmfocus: init at 1.0.2 #47662
wmfocus: init at 1.0.2 #47662
Conversation
Success on aarch64-linux (full log) Attempted: wmfocus Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: wmfocus Partial log (click to expand)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice feature! The buildInputs needs fixing, LGTM otherwise.
|
||
buildInputs = [python3 pkgconfig glib xorg.libX11 cmake cairo xorg.libX11]; | ||
nativeBuildInputs = [libxkbcommon xorg.libxcb xorg.xcbutilkeysyms]; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These buildInputs
and nativeBuildInputs
don't look right. nativeBuildInputs
should only contain packages that need to run at buildtime (here: pkgconfig
and python3
). The rest should be buildInputs
. Some of them, like cmake
aren't needed at all. I found this to be sufficient to build the package (and it seems to run fine):
nativeBuildInputs = [ python3 pkgconfig ];
buildInputs = [ cairo libxkbcommon xorg.xcbutilkeysyms ];
These buildInputs
also match the upstream list of dependencies (minus rust and cargo, which are implied by buildRustPackage).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xeji 2255419
Success on aarch64-linux (full log) Attempted: wmfocus Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: wmfocus Partial log (click to expand)
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)