Skip to content
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

Merged
merged 1 commit into from
Mar 20, 2015
Merged

luarocks support. #123

merged 1 commit into from
Mar 20, 2015

Conversation

xpol
Copy link
Contributor

@xpol xpol commented Mar 19, 2015

Add support for luarocks.

To release:

  1. bump version number in rockspec filename and file contents.
  2. tag v1.4.2 in github
  3. upload rockspec to https://rocks.moonscript.org

See Creating a rock

@creationix
Copy link
Member

Thanks. Two nits:

  • The License is actually Apache 2.0 to match Luvit
  • Can you squash your commits into one?

@xpol
Copy link
Contributor Author

xpol commented Mar 20, 2015

Done. 😄

creationix added a commit that referenced this pull request Mar 20, 2015
@creationix creationix merged commit d62cbfd into luvit:master Mar 20, 2015
@creationix
Copy link
Member

Does this look right? http://rocks.moonscript.org/modules/creationix/luv

@creationix
Copy link
Member

It doesn't seem to work, cmake is missing files. Maybe luarocks doesn't include git submodules?

@xpol
Copy link
Contributor Author

xpol commented Mar 20, 2015

Oh, yes.
The git submodule maybe not supported by luarocks. (I will discuss this with luarocks project.)

Is it possible that we switch from submodules to subtree ?
subtree has many advantages over submodules.

@xpol
Copy link
Contributor Author

xpol commented Mar 20, 2015

Added a issue here: luarocks/luarocks#334 .

@creationix
Copy link
Member

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?

@xpol
Copy link
Contributor Author

xpol commented Mar 20, 2015

Maybe we should add a cmake command to init and update the submodule if deps/libuv/src/uv.h is missing?

Good idea! 👍

@creationix
Copy link
Member

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.

d0664a6

@creationix
Copy link
Member

@joerg-krause
Copy link
Contributor

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.

@creationix
Copy link
Member

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.

@xpol
Copy link
Contributor Author

xpol commented Mar 24, 2015

Works fine! @creationix Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants