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

[New git version] v2.33.1 #3462

Closed
git-for-windows-ci opened this issue Oct 13, 2021 · 9 comments · Fixed by #3465
Closed

[New git version] v2.33.1 #3462

git-for-windows-ci opened this issue Oct 13, 2021 · 9 comments · Fixed by #3465
Assignees
Milestone

Comments

@git-for-windows-ci
Copy link

https://github.com/git/git/releases/tag/v2.33.1

@dscho
Copy link
Member

dscho commented Oct 13, 2021

Right. This was unexpected, and I had counted on a couple of weeks to be able to stabilize a couple of things:

I probably forgot more things that pose a particular challenge for upgrading Git for Windows to this Git version. With all that in mind, it might take me a while to get this release out.

@dscho
Copy link
Member

dscho commented Oct 13, 2021

  • I finished rotating my GPG key. This already led to a couple of hiccups in the automation and I fully expect more hiccups down the road.

To illustrate what I mean, have a look at this failure:

gpg: assuming signed data in '/tmp/tmp.iiV7YLCmLW/i686/git-for-windows.db.tar.xz'
gpg: Signature made Tue, Oct 12, 2021  6:28:16 PM GMT
gpg:                using RSA key 91883E11E83DC29D14104DB4EDD44359093056EE
gpg: checking the trustdb
gpg: marginals needed: 3  completes needed: 1  trust model: pgp
gpg: depth: 0  valid:   1  signed:   0  trust: 0-, 0q, 0n, 0m, 0f, 1u
gpg: next trustdb check due at 2026-03-22
gpg: Good signature from "Johannes Schindelin <[email protected]>" [ultimate]
Downloading current i686/git-for-windows.files.tar.xz...
gpg: assuming signed data in '/tmp/tmp.iiV7YLCmLW/i686/git-for-windows.files.tar.xz'
gpg: Signature made Tue, Oct 12, 2021  6:28:17 PM GMT
gpg:                using RSA key 91883E11E83DC29D14104DB4EDD44359093056EE
gpg: Good signature from "Johannes Schindelin <[email protected]>" [ultimate]
Downloading current i686/git-for-windows-mingw32.db.tar.xz...
gpg: assuming signed data in '/tmp/tmp.iiV7YLCmLW/i686/git-for-windows-mingw32.db.tar.xz'
gpg: Signature made Mon, Oct 11, 2021  3:52:54 PM GMT
gpg:                using RSA key 57759F748D223F034D8BE870BB3AA74136C569BB
gpg: Can't check signature: No public key
Could not verify GPG signature: /tmp/tmp.iiV7YLCmLW/i686/git-for-windows-mingw32.db.tar.xz
##[error]PowerShell exited with code '1'.

The problem here is that simply rotating the GPG key causes a failure while validatingof the current signature (which was done using the previous GPG key).

@dscho
Copy link
Member

dscho commented Oct 13, 2021

  • We had to rebuild the MSYS2 runtime

And of course this also had to cause issues: the git-sdk-64-minimal artifact (which we use to build Git and run its test suite in CI) now has an msys-2.0.dll that is incompatible with the one in C:\Program Files\Git\usr\bin, but the latter is picked up when running gpg.exe (due to the unfortunate setup of the build agents whereby all of Git for Windows' Unix tools are exposed via PATH). As a result, when trying to run the GPG-related tests, GPG would fail, but not when determining the prereq. So now I will have to fix that, too.

@dscho dscho self-assigned this Oct 13, 2021
@dscho
Copy link
Member

dscho commented Oct 13, 2021

My current plan is to wait for the Git artifacts run to pass, then to merge #3417.

After that, I hope that the final version of #3456 is ready to be merged. At that stage, I can start the final merging-rebase onto v2.33.1.

As mentioned before, all of this might spill over into tomorrow.

@dscho
Copy link
Member

dscho commented Oct 13, 2021

Here is the PR: #3465

@dscho
Copy link
Member

dscho commented Oct 13, 2021

And the Git artifacts run is here.

@dscho
Copy link
Member

dscho commented Oct 13, 2021

And my testing showed that we need one more thing. See #3465 (comment) for details.

@dscho dscho linked a pull request Oct 13, 2021 that will close this issue
@calle2010
Copy link

Thanks for all the efforts you put into Git for Windows!

I just wanted to leave here that the update from 2.33.0(2) to 2.33.1 contains two potentially breaking changes in dependencies:

The OpenSSH incompatibility can easily be worked around (see release notes). But for the LFS change I must wait for an upgrade of our Bitbucket server.

@dscho
Copy link
Member

dscho commented Oct 14, 2021

@calle2010 good point. Could you put up two discussions for those two issues? I would like to pin them, for visibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants