-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add Lua Rockspec package type #18
Conversation
dbf1121
to
bf13d6c
Compare
Pull Request Test Coverage Report for Build 7834942742
💛 - Coveralls |
I looks like over at https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#other-candidate-types-to-define, the candidate for LuaRocks packages is @wagoodman what do you think here? Should we change it to |
I went wity rockspec because it's to Lua what npm is to JavaScript. |
Maybe LuaRock would be better to differentiate it from the language? |
The only hints from the spec we get are:
and here:
There are some other issues in the purl spec repo that get close to this, but couldn't find any exact matches. I did find a PR comment that has an opinion here:
Here's my understanding from a purely lua point of view (anyone shout out if there is something I'm missing):
When making a new purl type, the vast majority seemed to be based around the packages that a package manager tool itself organizes (e.g.
There are some exceptions and inconsistencies:
|
Some specific thoughts relating to this PR:
So finally I think there are three candidates:
|
My vote would be for |
Signed-off-by: Laurent Goderre <laurent.goderre@docker.com>
bf13d6c
to
34a8785
Compare
Updated. |
Signed-off-by: Alex Goodman <wagoodman@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lots of thought to name things 😆 Thanks for the conversation on this 🙌
excellent... it would be great to also submit a PR for the spec to track this type and add tests entries so every tools can also use it. |
Adds the package manifest format for the Lua language: Rockspec. https://github.com/luarocks/luarocks/wiki/creating-a-rock
Ex: https://github.com/Kong/kong/blob/master/kong-3.7.0-0.rockspec