{"action":"created","issue":{"url":"https://api.github.com/repos/luvit/luv/issues/224","repository_url":"https://api.github.com/repos/luvit/luv","labels_url":"https://api.github.com/repos/luvit/luv/issues/224/labels{/name}","comments_url":"https://api.github.com/repos/luvit/luv/issues/224/comments","events_url":"https://api.github.com/repos/luvit/luv/issues/224/events","html_url":"https://github.com/luvit/luv/pull/224","id":139315988,"number":224,"title":"Use luarocks native build on rockspec","user":{"login":"tarruda","id":842846,"avatar_url":"https://avatars.githubusercontent.com/u/842846?v=3","gravatar_id":"","url":"https://api.github.com/users/tarruda","html_url":"https://github.com/tarruda","followers_url":"https://api.github.com/users/tarruda/followers","following_url":"https://api.github.com/users/tarruda/following{/other_user}","gists_url":"https://api.github.com/users/tarruda/gists{/gist_id}","starred_url":"https://api.github.com/users/tarruda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tarruda/subscriptions","organizations_url":"https://api.github.com/users/tarruda/orgs","repos_url":"https://api.github.com/users/tarruda/repos","events_url":"https://api.github.com/users/tarruda/events{/privacy}","received_events_url":"https://api.github.com/users/tarruda/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2016-03-08T16:02:20Z","updated_at":"2016-03-09T15:48:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/luvit/luv/pulls/224","html_url":"https://github.com/luvit/luv/pull/224","diff_url":"https://github.com/luvit/luv/pull/224.diff","patch_url":"https://github.com/luvit/luv/pull/224.patch"},"body":"This should make luv more compatible with luarocks echosystem:\r\n\r\n- Doesn't depend on cmake(which may not be available). luv is very simple to\r\n  compile, and cmake is used more to setup dependencies, but this is already\r\n  taken care of by luarocks.\r\n- Allows the user to specify custom libuv prefixes with LIBUV_DIR(which is the\r\n  idiomatic way of specifying dependencies location in luarocks)\r\n- Respects CC\r\n\r\nClose #223"},"comment":{"url":"https://api.github.com/repos/luvit/luv/issues/comments/194355626","html_url":"https://github.com/luvit/luv/pull/224#issuecomment-194355626","issue_url":"https://api.github.com/repos/luvit/luv/issues/224","id":194355626,"user":{"login":"tarruda","id":842846,"avatar_url":"https://avatars.githubusercontent.com/u/842846?v=3","gravatar_id":"","url":"https://api.github.com/users/tarruda","html_url":"https://github.com/tarruda","followers_url":"https://api.github.com/users/tarruda/followers","following_url":"https://api.github.com/users/tarruda/following{/other_user}","gists_url":"https://api.github.com/users/tarruda/gists{/gist_id}","starred_url":"https://api.github.com/users/tarruda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tarruda/subscriptions","organizations_url":"https://api.github.com/users/tarruda/orgs","repos_url":"https://api.github.com/users/tarruda/repos","events_url":"https://api.github.com/users/tarruda/events{/privacy}","received_events_url":"https://api.github.com/users/tarruda/received_events","type":"User","site_admin":false},"created_at":"2016-03-09T15:48:10Z","updated_at":"2016-03-09T15:48:10Z","body":"> For now can we simply point people to @tarruda's rockspec if they want a more minimal spec? I'm not familiar with rocks and how namespaces work. Is it under a different name or a different repository?\r\n\r\nTo install luv with the \"built\" rockspec one can simply invoke `luarocks build https://raw.githubusercontent.com/tarruda/luv/18bc34aa72492fddc558641b86ca6fdad5fd7589/luv-1.8.0-4.rockspec LIBUV_DIR...`, but it won't install as a separate package(the rock will still be registered as \"luv\"). It is possible to change the name in the rockspec, but AFAIK other modules that depend on luv won't be able to use it without breaking dependency on luv.\r\n\r\nHow about this: I rename the file to luv-1.8.0-4-nodeps.rockspec and both are maintained in luv tree(only changing the version number will be necessary, but this can be automated with a bump script). Publishing to luarocks is not necessary, projects that require this minimal version can install by pointing to the github raw content.\r\n\r\nTake neovim lua client for example, it depends on plain `luv`, and luarocks will install it using the default cmake rockspec. But if someone using neovim-client(eg: neovim) requires luv to be built using the \"nodeps\" rockspec, they can simply manually install luv prior to the client with: `luarocks build https://raw.githubusercontent.com/luv/luv/master/luv-1.8.0-4-nodeps.rockspec LIBUV_DIR...`\r\n\r\nIf this solution is acceptable I will fix this PR.\r\n\r\n"}}