-
Notifications
You must be signed in to change notification settings - Fork 39
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
Cannot clone MT32Emu/munt due to updated GitHub security #41
Comments
This is odd, because the .gitmodules file has the url as |
Ah, if you go to here you can see the old path. Edit: Although after manually editing and retrying, it still fails with the same error. The old git address must be somewhere else too. |
Go to Boxer/.git/modules/Vendor/Boxer-MT32Emu, and in the config file change:
to:
This should do the trick. |
Yes, that worked. Though I should mention that the modules folder is only created after trying to update the submodules once. |
This should be fixed by commit 5f209fd. |
When updating the submodules, cloning munt will fail:
Submodule 'munt' (git://github.com/munt/munt.git) registered for path 'Vendor/MT32Emu/munt'
Cloning into '/Users/xianmo/Downloads/Boxer/Vendor/MT32Emu/munt'...
fatal: remote error:
The unauthenticated git protocol on port 9418 is no longer supported.
Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.
fatal: clone of 'git://github.com/munt/munt.git' into submodule path '/Users/xianmo/Downloads/Boxer/Vendor/MT32Emu/munt' failed
The reason for the failure is updated security on GitHub which is described here.
I searched a bit for fixes, and I found this example from another project where they change the git path to github.
So if
git://github.com/munt/munt.git
is changed togithub:munt/munt.git
it might fix the issue.The text was updated successfully, but these errors were encountered: