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

Change source.url in rockspec to use git+https:// instead of git:// protocol (deprecated by GitHub) #270

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

milesrichardson
Copy link

@milesrichardson milesrichardson commented Mar 21, 2022

GitHub deprecated usage of git:// (unencrypted Git protocol) on March 15th, 2022, so LuaRocks will fail to install any package including a source.url starting with git://. According to the LuaRocks documentation, the correct syntax for referencing https:// URLs is git+https://.

This PR fixes source.url in the .rockspec file and bumps the version. I checked out the branch from master. I also made a second PR (attempting to) backport the fix to v0.16.x: #271

P.S. If your CI is breaking from luarocks packages using git://, here is a quick fix you can use:

git config --global url.https://github.com/.insteadOf git://github.com/

GitHub deprecated usage of `git://` (unencrypted Git protocol)
on March 15th, 2022.
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.

1 participant