Skip to content

Update Git to 2.22.0-rc1 [PR BUILD] #1209

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

Closed
wants to merge 4 commits into from

Conversation

derrickstolee
Copy link
Contributor

See microsoft/git#140 for the rebase to v2.22.0 (currently a release candidate)

@derrickstolee
Copy link
Contributor Author

Looks like PrefetchVerbWithoutSharedCacheTests.PrefetchBuildsIdxWhenMissingFromPrefetchPack() failed to regenerate a broken .idx file and the error message in the logs is here:

[2019-05-28 14:02:36 +00:00] Warning {"WarningMessage":"PrefetchStep: Git process IndexPackFile failed with errors: fatal: cannot read existing object info c2f10afa8c78a6356bc6d5cc7c69c3ea44b3be3f\n"}

I'll investigate to see if we are missing a read-object hook somewhere.

@@ -156,7 +156,7 @@ public static class Hooks
public const string PostCommandHookName = "post-command";
public const string ReadObjectName = "read-object";
public const string VirtualFileSystemName = "virtual-filesystem";
public const string PostIndexChangedName = "post-indexchanged";
public const string PostIndexChangedName = "post-index-change";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may require an on-disk layout upgrade for existing enlistments.

@derrickstolee
Copy link
Contributor Author

I'll re-open this when we are working on the full release again (and I should be able to drop the index-changed hook stuff).

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