-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
souffle: add darwin support #208373
souffle: add darwin support #208373
Conversation
1a49da6
to
19e971e
Compare
19e971e
to
e3e225d
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
patches = lib.optionals stdenv.isDarwin [ ./threads.patch ] ++ [ | ||
(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.
Please apply the patch unconditional
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.
Done. Also rebased and squashed.
e3e225d
to
885fe47
Compare
Description of changes
This PR fixes compilation error of souffle package for darwin. Currently is marked as broken.
Things done
After building the package in aarch64-darwin I was able to do a smoke test of the binary.
I also run the https://souffle-lang.github.io/simple example successfully
I am surprised that the changes introduced by
missing-override.patch
andhardeningDisable = lib.optionals stdenv.isDarwin [ "strictoverflow" ];
were not needed in linux already.The missing override patch is already present in upstream (See souffle-lang/souffle#2279) but is not released in 2.3. So I am applying it on linux also.
The other patches were done by consulting alicevision/geogram#2 (comment) and #39687 (comment)
Without threads.patch I was getting:
Without disabling strictoverflow I was getting
Without missing-override.patch I was getting
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes