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
lua-modules/generated: regen for formatting, luv -> 1.28 #61561
Conversation
@GrahamcOfBorg build luaPackages.luv neovim-unwrapped |
seems to be an issue for darwin https://github.com/NixOS/nixpkgs/pull/61561/checks?check_run_id=129210140 |
Hmm, it appears there are bundled copies of I've pushed a commit that addresses this (and ensures our system copy is what is used), but not especially fond of where that ended up. Suggestions/thoughts? |
weird it didn't appear before, is that a new thing in libuv ? I skipped over this as I thought it was darwin specific but it's for linux as well. I will have a look, check that it's fine with neovim and merge |
On Mon, 20 May 2019 21:06:57 -0700, Matthieu Coudron ***@***.***> wrote:
weird it didn't appear before, is that a new thing in libuv ? I skipped over this as I thought it was darwin specific but it's for linux as well. I will have a look, check that it's fine with neovim and merge
Perfect, thank you!
I seem to recall that yes this is a new problem from the update
and not a latent problem otherwise exposed or something.
(I think they did a bit of build-system reworking :))
…
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#61561 (comment) part: text/html
|
yeah the README mentions WITH_SHARED_LIBUV. Have you tried via cmakeFlags = [ -D WITH_SHARED_LIBUV ] ? (is there a specific error for " # This is a hacky way to specify -DWITH_SHARED_LIBUV=ON which # should be possible but I'm unable to make work." or it just doens't work xD ? ) |
@dtzWill I am doing some update on the lua infrastructure: I have fixed all (?) indentation issues in luarocks-nix and added some fixes that allow to generate packages for more luarock packages. |
I don't know if tried I'll poke at this a bit more right now and report... And hooray re:touchups on the luarocks-nix! \o/ |
Hmm can't seem to get the cmake flags through--although I admit that sure seems like an obtainable goal. I think unless you'd like to take a try then this is the best we have for now :). |
thanks for looking into it. Since I want to push some stuff behind this, let's merge it :p |
Motivation for this change
libuv is 1.28, bump luv to match, use improved formatting using current.
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)