Re: [PATCH 3/3] make --max-pack-size argument to 'git pack-object' count in bytes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Nicolas Pitre <nico@xxxxxxxxxxx> wrote:
> On Thu, 4 Feb 2010, Shawn O. Pearce wrote:
> > Nicolas Pitre <nico@xxxxxxxxxxx> wrote:
> > > >  --max-pack-size=<n>::
> > > > -	Maximum size of each output packfile, expressed in MiB.
> > > > -	The default is 4096 (4 GiB) as that is the maximum allowed
> > > > +	Maximum size of each output packfile.
> > > > +	The default is 4 GiB as that is the maximum allowed
> > > >  	packfile size (due to file format limitations). Some
> > > >  	importers may wish to lower this, such as to ensure the
> > > >  	resulting packfiles fit on CDs.
> > > 
> > > What file format limitation is alluded to here?  It has been a while 
> > > since the 4GB limit on pack file format has been removed.
> > 
> > The pack index v1 32 bit offset thing.  Which you fixed.
> > 
> > > If this is a 
> > > limitation of fast-import only then maybe this should be explained more 
> > > explicitly.
> > 
> > Damn.  It is.  fast-import can't write a v2 index.  Ugh.
> 
> Isn't it using write_idx_file()?  That function would do it all for you 
> already.

Nope.  Its still using its own code.  I never ported it to
write_idx_file().  And I don't have the CRC32 data available for
each object (it failed to compute/save it as it wrote).

I mean to port to write_idx_file() but didn't... :-(


-- 
Shawn.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]