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
rabbitmq_server: Don't use getconf optimization on non-Linux systems #40738
Conversation
This makes it impossible to build rabbitmq_server on Darwin, for example.
Success on x86_64-linux (full log) Attempted: rabbitmq_server Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: rabbitmq_server Partial log (click to expand)
|
Looks feasible; however I think the more sensible change is renaming |
@Profpatsch I'm not 100% clear on what you're proposing, can you show the changes needed? |
Here’s the first part of the necessary patch: master...Profpatsch:rabbitmq-getconf-darwin Can you look up which package your |
Is it okay to hold off on this? In staging, we have a getconf provided by unix-tools.nix: https://github.com/NixOS/nixpkgs/blob/staging/pkgs/top-level/unix-tools.nix#L50-L54 It should be a drop-in replacement, not needing the "small" hacks. unixtools.* should just be single binaries - no runCommand needed. |
Oh right, I forgot about that. @Ericson2314, do those tools work well with the cross system? That is, can we use the |
Yes. @matthewbauer's unixtools looks wonderful to me. |
Great, then I'm going to close this pull request in the hopes of better things to come. |
Yeah, on staging it’s already implemented, and rabbitmq fixed to the single-binary getconf. |
This makes it impossible to build rabbitmq_server on Darwin, for example.
Motivation for this change
Without it, I cannot build
rabbitmq_server
on macOS.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)