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
luaposix: 33.4.0 -> 34.0.4, add dependencies #35581
Conversation
@vyp please take a look, thanks! |
@Mic92 maybe you can take a look? Thanks! |
@GrahamcOfBorg build luajitPackages.bit32 lua52Packages.bit32 lua51Packages.bit32 lua51Packages.luaposix lua52Packages.luaposix luajitPackages.luaposix |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
It looks like darwin mixes multiple lua versions. |
@GrahamcOfBorg build lua52Packages.luaposix |
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
pkgs/top-level/lua-packages.nix
Outdated
}; | ||
|
||
buildPhase = '' | ||
cc -shared -I${lua}/include -Ic-api lbitlib.c -o bit32.so |
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.
Can you try to add lua
to buildInputs instead?
@GrahamcOfBorg build lua52Packages.bit32 |
Success on aarch64-linux (full log) Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
@GrahamcOfBorg build lua51Packages.bit32 |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
@GrahamcOfBorg build lua52Packages.bit32 |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
@GrahamcOfBorg build lua52Packages.luaposix |
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
@GrahamcOfBorg build luajitPackages.bit32 lua52Packages.bit32 lua51Packages.bit32 lua51Packages.luaposix lua52Packages.luaposix luajitPackages.luaposix |
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
pkgs/top-level/lua-packages.nix
Outdated
LUA="${lua}/bin/lua" \ | ||
LUA_INCDIR="-I${lua}/include" \ | ||
INST_LIBDIR="$out/lib/lua/${lua.luaversion}" \ | ||
INST_LUADIR="$out/share/lua/${lua.luaversion}" |
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 was able to remove the last 4 lines here. Are they really necessary?
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've got this options from rocks file https://github.com/luaposix/luaposix/blob/master/luaposix-git-1.rockspec#L33
It should work without it.
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.
mhm. Version should be there: https://github.com/luaposix/luaposix/blob/649454019dcc983eb6b398624e5b6c68d72f7808/lib/posix/version.lua.in
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.
nevermind it is still there.
}; | ||
|
||
buildInputs = [ perl ]; | ||
propagatedBuildInputs = [ std.normalize bit32 ]; |
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.
Is bit32
needed for lua5.2?
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.
Seems like luaposix library doesn't check if running under lua5.2, it loads bit32 lib every time. I would leave it.
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.
ok.
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.
removed those unneeded build options, it works.
Thanks for the review!
pkgs/top-level/lua-packages.nix
Outdated
LUA="${lua}/bin/lua" \ | ||
LUA_INCDIR="-I${lua}/include" \ | ||
INST_LIBDIR="$out/lib/lua/${lua.luaversion}" \ | ||
INST_LUADIR="$out/share/lua/${lua.luaversion}" |
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've got this options from rocks file https://github.com/luaposix/luaposix/blob/master/luaposix-git-1.rockspec#L33
It should work without it.
I rebased everything in 13c8e01 |
Motivation for this change
Refresh package and add new dependencies.
Thanks!
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)