Re: [PATCH 00/23] builtin-remote improvments

From: Jay Soffian <jaysoffian@gmail.com>
Date: Feb 24, 2009, 17:49 UTC
Message-ID: <76718490902240949i6ef70fc9i88b7a30a069ef374@mail.gmail.com>
In-Reply-To: <7v7i3fhgye.fsf@gitster.siamese.dyndns.org>

On Tue, Feb 24, 2009 at 12:39 PM, Junio C Hamano <gitster@pobox.com> wrote:

Show 8 quoted lines

> I don't mind pulling, as it is less work for me, but it would make
> harder for others to see the changes in flight and discuss, so...
>
> Even though I'd need to re-read them before merging them to 'next', I also
> think the earlier remote-set-head series and Jeff's changes on top of it
> looked more or less reasonable.  Perhaps I should rebase them on top of
> 'master' before moving them to 'next' and then you can build on top, which
> would reduce your new series by about 10 commits.

Putting Jeff's changes first reduces churn a small amount, as we don't introduce new code just to change it a few commits later in the same topic. So it keeps the history marginally cleaner. That was my thinking anyway. Otherwise you may as well just use what I sent last night and let me add one more patch on-top to clean-up that unused variable that gets passed to get_head_names().

Your call. :-)

j.