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
easystroke: init at 0.6.0-9-gf7c1614 #37353
Conversation
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.
Thanks 👍
I left some comments in the review.
enableParallelBuild = true; | ||
|
||
patches = [ | ||
(fetchpatch { |
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.
Can you explain why those patches are required? Did you attempt to submit them to upstream if they are an essential part of the package?
}; | ||
|
||
nativeBuildInputs = [ pkgconfig gcc7 wrapGAppsHook help2man ]; | ||
buildInputs = [ boost dbus_glib gnome3.gtkmm xorg.libXtst xorg.inputproto |
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.
Can you fix the indention & order here?
Sorting dependencies alphabetically and having a common indention level across all list elements is what most people seem to do. Unfortunately, there is no style guide (yet).
let | ||
version = "0.6.0-9-gf7c1614"; | ||
in stdenv.mkDerivation { | ||
name = "easystroke-${version}"; |
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.
According to our naming guide (https://nixos.org/nixpkgs/manual/#sec-package-naming) the package name should probably be (final form after evaluation): easystroke-unstable-2000-01-31
(with correct date)
@@ -0,0 +1,55 @@ | |||
{ stdenv, fetchFromGitHub, fetchpatch, pkgconfig, boost, dbus_glib, gnome3, |
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.
Why did you specify gcc7
there (& within nativeBuildInputs)? Usually sticking with just using whatever stdenv
provides is what you want.
@GrahamcOfBorg build easystroke |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: easystroke Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: easystroke Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: easystroke Partial log (click to expand)
|
This commit is based on work by Andreas Schaefer in NixOS#37353
This pull request has been mentioned on Nix community. There might be relevant details there: |
Replaced by #52424. |
Motivation for this change
easystroke: init at 0.6.0-9-gf7c1614
Easystroke is a X11 gesture recognition application.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)