Re: Interim maintainer tree

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 4, 2009, 09:54 UTC
Message-ID: <7viqevpjr3.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20091004064129.GB7076@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 5 quoted lines

> For my stuff, I think you can just take over my tips. I was just
> collecting and polishing topics, while Shawn was doing integration. For
> topics I have, please use my versions rather than applying from the
> list; many of them have extra fixes squashed in, acked-by's collected
> from the list, etc.

Many topics Shawn merged to his 'pu' were identical to what you have, except for mr/gitweb-snapshot. The patch-ids in the series are the same but you have Jakub's Ack added, so I'll pick yours up and rebuild.

> Here's a brief status report on what's in my tree:
>
>   proposed-master
>     One-off patches that I think should go straight to master.

I briefly looked at them after looking at what Shawn queued to his 'master'; they all looked sane.

Show 24 quoted lines

>   ef/mscv-noreturn
>     This is the latest round and I think should be ready for at least
>     'next' (maybe even 'master' as it is really about the build and not
>     about functionality).
>
>   ef/msys-imap
>     This is from an RFC which has generated some comments. He should be
>     posting another round soon. 'pu' at best.
>
>   fc/mutt-alias
>     Latest round that addressed comments. Ready for 'next' if not
>     'master'.
>
>   jn/gitweb-patch
>     After some comments with Jakub, I think the code is right but he
>     promised a re-roll with more in the commit message.
>
>   mr/gitweb-snapshot
>     This is probably the pu rewind you saw. He posted a v5 of his
>     series. I didn't look at it closely, but Jakub ack'd it.
>
>   tf/doc-pt-br
>     Minor translation update, ack'd by somebody who can read it. :)
>     Ready for 'master'.

I've queued all of these to 'pu' for tonight as I haven't still fully recovered yet from jetlag, but I agree with the above assessment.

Show 8 quoted lines

> There are a few patches floating on the list that I haven't picked up or
> looked too closely at yet. Just so we don't drop anything, they are:
>
>   - curl http auth tweak:
>     http://article.gmane.org/gmane.comp.version-control.git/129455
>     The author said he didn't really test it, and I haven't set up http
>     auth to test it with, but probably somebody should do so before
>     applying. :)

Heh ;-)

>   - a new rev-cache from Nick

What Shawn had in his 'pu' had one patch removed from what I had earlier (perhaps my 'tip list' was faulty). I'll pick the new one up later.

>   - silence gcc warning:
>     http://article.gmane.org/gmane.comp.version-control.git/129485
>     The warning is overly cautious, I think, but it is a dubious enough
>     construct that it is probably worth fixing.

I cannot reach gmane now, but if this is about -Wextra, I'd rather not touch it before the release. "comparison between signed and unsigned" tends to be excessive and IMNSHO it is crazy to use -Wextra and -Werror together.

>   - enable openssl on msvc
>     http://article.gmane.org/gmane.comp.version-control.msysgit/7238
>     This goes on top of ef/msys-imap, but I think that will be getting a
>     re-roll.

Ok.

Thanks.