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
consul-ui: fix build #41268
consul-ui: fix build #41268
Conversation
Update the dependencies used for building consul-ui. therubyracer is no longer required and fails to build. Instead, a nodejs binary is required to build the assets. Patch out unnecessary executions of `bundle` which can cause failures.
@7c6f434c Looks like there might be an issue with your latest commit: https://gist.github.com/GrahamcOfBorg/27d3550cdabe42098a41f92d433a4181 |
@GrahamcOfBorg eval |
A direct reference stopped on |
@GrahamcOfBorg build consul-ui |
Success on x86_64-linux (full log) Attempted: consul-ui Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: consul-ui Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: consul-ui Partial log (click to expand)
|
Motivation for this change
consul-ui is currently failing to build in 18.03 and master (#41238).
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)My previous PR (#41239) was against the 18.03 branch because the fix did not work in master. This newer version works in both master and 18.03.
Update the dependencies used for building consul-ui. therubyracer is no longer required and fails to build. Instead, a nodejs binary is required to build the assets. Patch out unnecessary executions of
bundle
which can cause failures and also prevents need for patching the Gemfile.lock in the consul source, based on suggestion by @nh2.Closes #41238