* BUG! missing .idx causes .pack to be removed
@ 2010-08-05 17:01 Enrico Weigelt
2010-08-06 3:24 ` Tay Ray Chuan
0 siblings, 1 reply; 4+ messages in thread
From: Enrico Weigelt @ 2010-08-05 17:01 UTC (permalink / raw)
To: git
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
----------------------------------------------------------------------
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: BUG! missing .idx causes .pack to be removed
2010-08-05 17:01 BUG! missing .idx causes .pack to be removed Enrico Weigelt
@ 2010-08-06 3:24 ` Tay Ray Chuan
2010-08-06 7:14 ` René Scharfe
0 siblings, 1 reply; 4+ messages in thread
From: Tay Ray Chuan @ 2010-08-06 3:24 UTC (permalink / raw)
To: weigelt; +Cc: git
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: BUG! missing .idx causes .pack to be removed
2010-08-06 3:24 ` Tay Ray Chuan
@ 2010-08-06 7:14 ` René Scharfe
2010-08-13 16:02 ` Enrico Weigelt
0 siblings, 1 reply; 4+ messages in thread
From: René Scharfe @ 2010-08-06 7:14 UTC (permalink / raw)
To: Tay Ray Chuan; +Cc: weigelt, git
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é
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: BUG! missing .idx causes .pack to be removed
2010-08-06 7:14 ` René Scharfe
@ 2010-08-13 16:02 ` Enrico Weigelt
0 siblings, 0 replies; 4+ messages in thread
From: Enrico Weigelt @ 2010-08-13 16:02 UTC (permalink / raw)
To: git
* 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
----------------------------------------------------------------------
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-08-13 16:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-05 17:01 BUG! missing .idx causes .pack to be removed Enrico Weigelt
2010-08-06 3:24 ` Tay Ray Chuan
2010-08-06 7:14 ` René Scharfe
2010-08-13 16:02 ` Enrico Weigelt
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).