-
Notifications
You must be signed in to change notification settings - Fork 183
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
luarocks support. #123
luarocks support. #123
Conversation
Thanks. Two nits:
|
Done. 😄 |
Does this look right? http://rocks.moonscript.org/modules/creationix/luv |
It doesn't seem to work, cmake is missing files. Maybe luarocks doesn't include git submodules? |
Oh, yes. Is it possible that we switch from submodules to subtree ? |
Added a issue here: luarocks/luarocks#334 . |
I'm not going to switch to subtree. I know the tradeoffs and prefer submodules for this project. I did try manually creating the .src.rock file with the libuv submodule included (the packer didn't include it automatically so I added it to the zip), but when I tried to install, it ignored my uploaded zip file and did a non-recursive clone anyway. Maybe we should add a cmake command to init and update the submodule if deps/libuv/src/uv.h is missing? |
Good idea! 👍 |
Doesn't help. I read somewhere that luarocks deletes the .git folder after cloning. If this is true, the submodule update command I added will have no effect. |
How about providing an uploaded release tarball instead of a tag like it's done for luvi? Note, I don't mean the automatically generated tarball by github, but an uploaded one. This has some advantages for package maintainers and build systems. I would also suggest to use a proper release tarball name in case of luvi: -, e.g. luvi-v1.0.1.tar.gz, luv-v1.4.2.tar.gz. Using a release tarball there is no need for any git commands in a Makefile. A github user can still follow the "Building From Source" section to get a working copy, a package maintainer or build system can download the release tarball. |
Ok, I manually created a github release with a hand-crafted tarball. This seems to work now. Try with luv 1.4.2-2 on luarocks. |
Works fine! @creationix Thank you! |
Add support for luarocks.
To release:
v1.4.2
in githubSee Creating a rock