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
lumail: 2.9 -> 3.1 #32710
lumail: 2.9 -> 3.1 #32710
Conversation
258d56d
to
7358936
Compare
Tested and works for me. Please review the package, I made some serious modifications. @orivej |
Please don't merge. Found a bug. The lua libs seem not to be loaded. I cannot use "loadstring" for example. Something is wrong. |
I'm currently trying to set the lua path. I fail hard. |
|
Found the issue: lumail/lumail#348 I will squash the patches if someone approves. |
#36816 documents how to use lua 5.2 This branch uses 5.1, which is the simplest fix for the issue. If requested, I can change the package to use 5.2 (or 5.3, if requested of course). |
@GrahamcOfBorg build lumail |
No attempt on x86_64-darwin The following builds were skipped because they don't evaluate on x86_64-darwin: lumail No log is available. |
Success on aarch64-linux (full log) Attempted: lumail Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: lumail Partial log (click to expand)
|
Nice. I'll do a squash! |
Includes: * Package gets a flag to use the debug build * install phase installs all lua scripts from the package and makes lumail find them * global configuration which is shipped with the package can be overridden, if desired * parallel building enabled
This is necessary because the standard library which is distributed with lumail (the lumail core configuration so to speak) is written for lua5.1 apparently. The website states 5.1 or 5.2 or 5.3, but 5.2 fails because "loadstring" was deprecated in lua 5.2. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
a1e8a69
to
13e95f3
Compare
Ready for merging, IMO. |
Motivation for this change
Wanna try out lumail
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)This fails for me with
I assume that the
makeFlags
do not properly work. NeitherDESTDIR
norPREFIX
is overriden, as the log suggests.