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
yara: 3.10.0 -> 3.11.0, fix build #73164
Conversation
We have build problems again: make[1]: *** No rule to make target 'libyara/.libs/libyara.a', needed by 'yara'. Stop.
make[1]: Leaving directory '/build/source'
make: *** [Makefile:1062: all-recursive] Error 1 I could use the patch that was used before, but I don’t think this is the best idea based on this issue: out-of-bounds read, write in yr_execute_code. What are some ideas for update this package? |
You could reuse the patch, but I think this would be simpler and hopefully upstream can fix VirusTotal/yara#1036. |
Thank you so much! |
Ooh, it looks like some magic happened and upstream acted pretty quick Perhaps you could try the above patch and see if it makes worldofpeace@7d0dd7b unneeded. |
pkgs/tools/security/yara/default.nix
Outdated
# TODO: This patch should not be necessary in the next release | ||
patches = [ | ||
(substituteAll { | ||
src = ./fix-issue-1036.patch; |
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.
Github has a pretty cool feature, you can get a generated patch from a url
(fetchpatch {
url = "https://github.com/VirusTotal/yara/commit/04df811fa61fa54390b274bfcf56d7403c184404.patch";
sha256 = stdenv.lib.fakeSha256; # replace
})
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.
Wow! Thanks, you surprised me!
I was a little wrong with the hash for the patch. All is well now. |
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.
LGTM, builds. Thanks 🌟
Motivation for this change
Update YARA 3.10.0 -> 3.11.0
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @