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
vim_configurable: unpin from old lua 5.1 #40749
Conversation
Success on aarch64-linux (full log) Attempted: vim Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: vim Partial log (click to expand)
|
This pin was added in 2014 during a lua upgrade, but it appears vim today builds fine with lua 5.2, so we can have it just depend on the `lua` package.
Success on x86_64-linux (full log) Attempted: vim_configurable Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: vim_configurable Partial log (click to expand)
|
let's check the Darwin build |
Success on x86_64-darwin (full log) Attempted: vim_configurable Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: vim_configurable Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: vim_configurable Partial log (click to expand)
|
This broke |
Thanks @orivej for catching this. The cause seems to be that I'll pin |
pinned |
vimNox is currently built from a patched vim 7.3 source from 2012 that requires lua 5.1. Temporary fix, see discussion in #40749.
This pin was added in 2014 during a lua upgrade, but it appears vim today builds
fine with lua 5.2, so we can have it just depend on the
lua
package.Motivation for this change
Noticed while building that lua 5.1 is broken (#40748) so I tried just using lua. Seems to work fine.
Things done
I'm not sure why callPackage wasn't sufficient to pass this a lua, but with the inherit it's clearly picking up the unpinned
lua
package as intendedbuild-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)