Skip to contentSkip to conversation

am --abort on null cache entries fails

BUG? Stephen Boyd 2 participants 2 messages Jun 15, 2016

lore
Share & tools
lore
#0 lore raw

Hi,

I was trying git am -3 with a patch that touched files that didn't exist in the branch I was on. Obviously it failed badly, so I wanted to abort out of the git am state with git am --abort. Unfortunately, it seems that git am --abort in this scenario fails with this error:

error: cache entry has null sha1: <non-existant-file>

and then leaves the file in my working tree untracked. This didn't used to happen, so I bisected it down to this commit

commit 4337b5856f88f18da47c176e3cbc95a35627044c
Author: Jeff King <[email protected]>
Date: Sat Jul 28 11:05:24 2012 -0400
do not write null sha1s to on-disk index

Which definitely introduced that error message. How do we fix this?

Signature · 2 lines
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
#1 lore raw
Quoted 10 lines · Stephen Boyd

I was trying git am -3 with a patch that touched files that didn't exist in the branch I was on. Obviously it failed badly, so I wanted to abort out of the git am state with git am --abort. Unfortunately, it seems that git am --abort in this scenario fails with this error:

error: cache entry has null sha1: <non-existant-file>

and then leaves the file in my working tree untracked. This didn't used to happen, so I bisected it down to this commit

Quoted 6 lines

commit 4337b5856f88f18da47c176e3cbc95a35627044c
Author: Jeff King <[email protected]>
Date: Sat Jul 28 11:05:24 2012 -0400
do not write null sha1s to on-disk index

Which definitely introduced that error message.

Yep. It's a bug, but that commit does not introduce it; it actually just notices the bug earlier.

Quoted 1 line

How do we fix this?

See this thread for the current discussion and some possible fixes:

http://thread.gmane.org/gmane.comp.version-control.git/217172

-Peff