-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
eztrace: 1-1-7 -> 1.1-11, fix build, switch to fetchFromGitLab, refactor #144645
Conversation
b7e684c
to
23448e8
Compare
license = lib.licenses.cecill-b; | ||
maintainers = with lib.maintainers; [ ]; | ||
license = licenses.cecill-b; | ||
maintainers = with maintainers; [ ]; |
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.
You know what I'm going to suggest...
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.
That I add myself as a maintainer? I honestly don't know
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.
If you're interested, yeah, I think it would be good to add yourself as maintainer.
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.
I'm not. Only doing this because of ZHF 🤷
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.
Fair enough 🙂 .
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.
WFM nixos x86_64
Yup, think it's sensible grabbing the gitlab source in this case.
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.
Result of nixpkgs-review pr 144645
run on x86_64-linux 1
1 package built:
- eztrace
Motivation for this change
ZHF: #144627
https://hydra.nixos.org/build/157349557
I do realize that release tarballs are preferred over source tarballs of the release - but having to patch the resulting configure script feels unnecessary, when we can generate the makefile and configure script ourselves.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)