Re: [PATCH/RFC] Allow writing loose objects that are corrupted in a pack file

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Nicolas Pitre
Date: Wednesday, January 7, 2009 - 8:31 am

On Tue, 6 Jan 2009, R. Tyler Ballance wrote:


As long as those files are not within the .git directory that should be 
fine.


Problem is that none of that "makes sense".  If a real git corruption 
was there, it wouldn't disappear without explicit action from your part.  
What git v1.6.1 is able to do over earlier versions is to still function 
properly if some corrupted objects have a redundant copy in the same 
repository, but it wouldn't stop complaining about the existence of 
corrupted data.  Doing a 'git gc' or 'git repack -a' might get rid of 
the corruption.  And a 'git pull' might hide it if the received pack 
during the pull operation happens to contain another copy of the object 
that was corrupted before, but it wouldn't prevent 'git fsck --full' 
from seeing it.

The fact that you cannot reproduce the corruption issues after 
unarchiving a repository that was known to have problems right before it 
was archived is really really strange.  That does not rule out git bugs 
of course, but at least this shows that no actual corruption on disk was 
initially involved.

Again, I'd suggest you perform your git usage within a script session so 
to capture the exact operation performed and error messages produced 
when/if similar problems do occur again.  Otherwise we're only running 
after our tail.


Nicolas
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH/RFC] Allow writing loose objects that are corru ..., R. Tyler Ballance, (Wed Jan 7, 12:41 am)
Re: [PATCH/RFC] Allow writing loose objects that are corru ..., Nicolas Pitre, (Wed Jan 7, 8:31 am)
Re: Public repro case! Re: [PATCH/RFC] Allow writing loose ..., Boyd Stephen Smith Jr., (Wed Jan 7, 7:52 pm)
Re: [PATCH] Wrap inflateInit to retry allocation after rel ..., R. Tyler Ballance, (Thu Jan 8, 11:15 am)