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
elfio: init at 3.8 #110322
elfio: init at 3.8 #110322
Conversation
2bb04d5
to
1973091
Compare
|
||
phases = [ "unpackPhase" "installPhase" ]; |
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.
phases = [ "unpackPhase" "installPhase" ]; |
Please never overwrite phases. This breaks many of the hooks used inside nixpkgs.
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.
Okay. Removed it. Should I add dontBuild
or dontConfigure
then?
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.
Yes, please.
1973091
to
27e5228
Compare
I marked this as stale due to inactivity. → More info |
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.
Please retest if the package still works.
mkdir -p $out/include | ||
cp -R ./elfio $out/include |
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.
mkdir -p $out/include | |
cp -R ./elfio $out/include | |
mkdir -p $out/include | |
cp -R ./elfio $out/include |
Another elfio expr was added to nixpkgs in the meantime: 3e231f0 |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)