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
lief: init at 0.9.0 #42565
lief: init at 0.9.0 #42565
Conversation
Success on x86_64-darwin (full log) Attempted: lief Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: lief Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: lief 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.
Please add meta
with the usual attributes and make sure platforms
matches the prebuilt binary libs in the tarball.
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: lief Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: lief Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: lief Partial log (click to expand)
|
license = [ licenses.asl20 ]; | ||
platforms = platforms.linux; | ||
maintainers = [ maintainers.lassulus ]; | ||
}; |
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.
meta
belongs outside the fetchzip
, not in it.
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.
This use is definitely okay. fetchzip output can be installed as a normal derivation so it is pretty much needed to use meta here. I also much prefer it to the all too common cp -r $src/* $out
because it will be fixed output.
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.
Sorry, didn't know about that. Never seen this use before.
https://lief.quarkslab.com/
For now I just use the released version because I couldn't get the cmake build to work, as it wants to do something with git I couldn't figure out.