-
Notifications
You must be signed in to change notification settings - Fork 32
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
perl: update to 5.38.0 #115
Conversation
I guess this PR gives a good reason to switch Last time we had a major Perl version upgrade was in #57, and I seem to have failed to record which dependencees needed to be rebuilt. But there is git-for-windows/git-sdk-64@1fb28b3 which lists these:
So we'll also need to increment the respective |
Co-authored-by: Matthias Aßhauer <[email protected]> Signed-off-by: Johannes Schindelin <[email protected]>
Let's use the excuse of having to rebuild the Perl module after upgrading to a new Perl version anyway. Signed-off-by: Johannes Schindelin <[email protected]>
Ugh. Since msys2/MSYS2-packages@9652bfd, |
Let's use the excuse of having to rebuild the Perl module after upgrading to a new Perl version anyway. Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
I'm happy to remove the perl dep (somehow?), I just saw it linking in CI and made it explicit. Not a vim user, so not sure what would be lost there. |
It was not actually your change that caused the perl DLL to be linked statically into |
Let's close this in favor of #139. |
This closes git-for-windows/git#4488