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
zookeeper: Fix wrong paths in startup script #70314
Conversation
ping |
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.
i did find a /bin/echo on the start command, please take a look at my comments :)
pkgs/servers/zookeeper/default.nix
Outdated
@@ -19,14 +19,14 @@ stdenv.mkDerivation rec { | |||
mkdir -p $out/bin | |||
cp -R bin/{zkCli,zkCleanup,zkEnv,zkServer}.sh $out/bin | |||
patchShebangs $out/bin | |||
substituteInPlace $out/bin/zkServer.sh \ | |||
--replace /bin/echo ${coreutils}/bin/echo \ | |||
--replace "/usr/bin/env bash" ${bash}/bin/bash |
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 one should be fine, and I don't see it in the script
--replace "/usr/bin/env bash" ${bash}/bin/bash |
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.
I just blindly copied that, but you are right, this is not necessary; I have removed it. :-)
492cda7
to
e1e2be1
Compare
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.
nix-review
passes on NixOS
diff LGTM
patch LGTM
[1 built]
https://github.com/NixOS/nixpkgs/pull/70314
5 package were build:
perl528Packages.NetZooKeeper perl530Packages.NetZooKeeper zkfuse zookeeper zookeeper_mt
Motivation for this change
Substitution needs to be done before wrapping, else it doesn't substitute anything.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @