BUG! missing .idx causes .pack to be removed

BUG! missing .idx causes .pack to be removed

[Enrico Weigelt]

Hi folks,


while doing some experiments I've removed the *.idx files and 
ran git-repack - that complained about missing objects and removed
all *.pack files !

I'm sure that's a bug - an ugly one ;-o


cu
-- 
----------------------------------------------------------------------
 Enrico Weigelt, metux IT service -- http://www.metux.de/

 phone:  +49 36207 519931  email: weigelt@metux.de
 mobile: +49 151 27565287  icq:   210169427         skype: nekrad666
----------------------------------------------------------------------
 Embedded-Linux / Portierung / Opensource-QM / Verteilte Systeme
----------------------------------------------------------------------

Re: BUG! missing .idx causes .pack to be removed

[Tay Ray Chuan]

Hi,

On Fri, Aug 6, 2010 at 1:01 AM, Enrico Weigelt <weigelt@metux.de> wrote:
> while doing some experiments I've removed the *.idx files and
> ran git-repack - that complained about missing objects and removed
> all *.pack files !
>
> I'm sure that's a bug - an ugly one ;-o

IIRC, a .pack file is useless without its corresponding .idx file. So
the removal of a .pack file makes sense here - to me, at least.

-- 
Cheers,
Ray Chuan

Re: BUG! missing .idx causes .pack to be removed

[René Scharfe]

Am 06.08.2010 05:24, schrieb Tay Ray Chuan:
> Hi,
> 
> On Fri, Aug 6, 2010 at 1:01 AM, Enrico Weigelt <weigelt@metux.de> wrote:
>> while doing some experiments I've removed the *.idx files and
>> ran git-repack - that complained about missing objects and removed
>> all *.pack files !
>>
>> I'm sure that's a bug - an ugly one ;-o
> 
> IIRC, a .pack file is useless without its corresponding .idx file. So
> the removal of a .pack file makes sense here - to me, at least.

git-index-pack can rebuild the index for a pack file.  Not sure if it
should be done automagically, but the file is not completely useless.

René

Re: BUG! missing .idx causes .pack to be removed

[Enrico Weigelt]

* René Scharfe <rene.scharfe@lsrfire.ath.cx> wrote:
> > IIRC, a .pack file is useless without its corresponding .idx file. So
> > the removal of a .pack file makes sense here - to me, at least.
> 
> git-index-pack can rebuild the index for a pack file.  Not sure if it
> should be done automagically, but the file is not completely useless.

ACK. packfiles are completely self-contained - the index files are
only used for lookup and can be created any time (IMHO that's also
done automatically on remote transfers). So removing a packfile,
just because the index is missing, causes an serious data loss.
That's what I'd consider a critical bug!

The correct solution would be either recreating the index
automatically or at least spit out an big-fat warning.


cu
-- 
----------------------------------------------------------------------
 Enrico Weigelt, metux IT service -- http://www.metux.de/

 phone:  +49 36207 519931  email: weigelt@metux.de
 mobile: +49 151 27565287  icq:   210169427         skype: nekrad666
----------------------------------------------------------------------
 Embedded-Linux / Portierung / Opensource-QM / Verteilte Systeme
----------------------------------------------------------------------