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

build: improve rockspec #391

Merged
merged 1 commit into from
Aug 1, 2022
Merged

build: improve rockspec #391

merged 1 commit into from
Aug 1, 2022

Conversation

teto
Copy link
Contributor

@teto teto commented Jul 30, 2022

the data folder was not copied. Also the 'builtin' type will copy automatically everyhting that is in the lua folder

@teto
Copy link
Contributor Author

teto commented Jul 31, 2022

Looks ok to me. What would be perfect would be to upload a new version on luarocks.org

@teto teto marked this pull request as ready for review July 31, 2022 16:57
the data folder was not copied. Also the 'builtin' type will copy automatically everyhting that is in the lua folder
@DieracDelta
Copy link

DieracDelta commented Jul 31, 2022

FWIW with Nix this branch fixes telescope's failure to syntax highlight. +1 for merging. Example usage

@Conni2461
Copy link
Collaborator

Thanks :)

@Conni2461 Conni2461 merged commit 31807ee into nvim-lua:master Aug 1, 2022
@Conni2461
Copy link
Collaborator

do i need to do something to publish a newer version? uploade says teto here: https://luarocks.org/modules/teto/plenary.nvim

@teto
Copy link
Contributor Author

teto commented Aug 1, 2022

@Conni2461 I've removed my package from luarocks.org so now plenary.nvim developers are in charge !

EDIT: what would be cool is to add the "neovim" label to the package (from the luarocks.org interface). It should be picked up from the rockspec but in my experience, that was not the case.

EDIT2: another example how some lua developers version their rockspecs: https://github.com/starwing/lua-protobuf/tree/master/rockspecs

@teto teto deleted the fix-rockspec branch August 2, 2022 11:53
@teto
Copy link
Contributor Author

teto commented Aug 9, 2022

@Conni2461 sorry to poke you again, there are a few rockspecs I wanna install that lists plenary as a dependency but now that I removed it from luarocks I an't install those anymore. Do you think you could upload the plenary rockspec to the root manifest on luarocks.org or should I reupload it myself until you or another maintainer get time ?

@Conni2461
Copy link
Collaborator

I'll upload one tomorrow. Sorry i am currently really busy and it slipped my mind. Thanks for poking again :)

@Conni2461
Copy link
Collaborator

okay its uploaded: https://luarocks.org/modules/Conni2461/plenary.nvim but i can't seem to get luarocks to read any of my organizations so that nvim-lua is the owner of the rockspec. Do you know if that is possible? i found a open resty "org" https://luarocks.org/modules/openresty but i am not sure if this is even a real github org that was used or if they just created a openresty luarocks account.

@teto
Copy link
Contributor Author

teto commented Aug 10, 2022

I dont think a team can own a rockspec. Probably for the same reasons you can't generate a github API token for a github org. There may be a limitation on the luarocks side that prevents several accounts to own a rockspec, one would need to doublecheck (on their gitter maybe). In my settings I can link different github accounts to my profile so probably one can create a shared account on luarocks and link different org members github account into it. I will ask for more information.

@teto
Copy link
Contributor Author

teto commented Aug 23, 2022

Indeed it's not possible (yet):
luarocks/luarocks-site#127
luarocks/luarocks-site#173

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.

None yet

3 participants