threads / patch / 7569

patch, 8 partsgit-repack --max-pack-size

Subject: [PATCH 0/8] git-repack --max-pack-size

## tl;dr

4 messages between Apr 8, 2007 and Apr 9, 2007. Diffs are folded; open one to read it.

replies: 3people: 2as markdown or json

Dana How· Apr 8, 2007, 23:17 UTC · lore

The three most common ways of making large packfiles are git-fast-import, the first git-repack, or git-repack -a. The first already supports a "--max-pack-size=N" option, which limits the resulting packfiles to N megabytes. This patchset adds the same option, with the same behavior, to git-repack to handle the other two cases.

After reviewing others' comments, this latest patchset touches 2/3 fewer files and 1/6 fewer lines, and introduces no behavior not seen elsewhere in git.

Nicolas Pitre· Apr 9, 2007, 02:33 UTC · re: Dana How · lore

Re: [PATCH 0/8] git-repack --max-pack-size

On Sun, 8 Apr 2007, Dana How wrote:
> After reviewing others' comments,  this latest patchset
> touches 2/3 fewer files and 1/6 fewer lines,  and
> introduces no behavior not seen elsewhere in git.
I'm starting to like it, but...

I have some extensive changes that touch the same area more or less. And so far I think it might be easier to rebase your changes on top of mine rather than the other way around.

So I'd like if my changes would be merged first and then yours to be applied afterwards. They still have some issues left anyway.

I'm trying to format my changes now and send them tonight.
Nicolas
Dana How· Apr 9, 2007, 18:54 UTC · re: Nicolas Pitre · lore

Re: [PATCH 0/8] git-repack --max-pack-size

On 4/8/07, Nicolas Pitre <nico@cam.org> wrote:
Show 16 quoted lines
> On Sun, 8 Apr 2007, Dana How wrote:
>
> > After reviewing others' comments,  this latest patchset
> > touches 2/3 fewer files and 1/6 fewer lines,  and
> > introduces no behavior not seen elsewhere in git.
>
> I'm starting to like it, but...
>
> I have some extensive changes that touch the same area more or less.
> And so far I think it might be easier to rebase your changes on top of
> mine rather than the other way around.
>
> So I'd like if my changes would be merged first and then yours to be
> applied afterwards.  They still have some issues left anyway.
>
> I'm trying to format my changes now and send them tonight.

Thanks for sending them out. I guess I'll update mine to apply after yours once yours show up in the appropriate branch.

Thanks,
-- 
Dana L. How  danahow@gmail.com  +1 650 804 5991 cell
Dana How· Apr 9, 2007, 19:43 UTC · re: Dana How · lore

Re: [PATCH 0/8] git-repack --max-pack-size

On 4/8/07, Dana How <danahow@gmail.com> wrote:
Show 10 quoted lines
> The three most common ways of making large packfiles
> are git-fast-import, the first git-repack, or git-repack -a.
> The first already supports a "--max-pack-size=N" option,
> which limits the resulting packfiles to N megabytes.
> This patchset adds the same option, with the same
> behavior, to git-repack to handle the other two cases.
>
> After reviewing others' comments,  this latest patchset
> touches 2/3 fewer files and 1/6 fewer lines,  and
> introduces no behavior not seen elsewhere in git.
At the moment,  I plan the following changes to this patchset:
* Redo after 64b index applied (Nicolas)
* Bad re-use of header around sha1write (Nicolas)
* Set object_entry.offset before write_object (Dana)
* Finish reducing struct object_entry (Junio)
* Remove spaces from inside () (Junio)

Concerning refactorings, I'd prefer to keep those in a separate (follow-on?) patchset.

Thanks,
-- 
Dana L. How  danahow@gmail.com  +1 650 804 5991 cell

← back to recent threads