From: Daniel Barkalow Date: Fri, 29 Feb 2008 02:17:43 GMT Subject: Re: [PATCH 0/5] Builtin "git remote" Message-ID: In-Reply-To: <7vwsoofrue.fsf@gitster.siamese.dyndns.org> On Thu, 28 Feb 2008, Junio C Hamano wrote: > Johannes Schindelin writes: > > > It also demonstrates that format-patch --cover-letter should default to -M > > for the diffstat (search for "git-remote.perl"...). > > It probably should inherit the settings for the primary part. > You did generate the series with -M, especially [PATCH 4/5], > didn't you? Obviously not the output_format, but everything else, I assume? Easy enough to pass the main diff options into make_cover_letter() and use them to initialize the diff options there. > > Johannes Schindelin (5): > > path-list: add functions to work with unsorted lists > > parseopt: add flag to stop on first non option > > Test "git remote show" and "git remote prune" > > Make git-remote a builtin > > builtin-remote: prune remotes correctly that were added with --mirror > > Also I wish the default indentation were not that deep, and also > line-wrapped at around 72-76 columns. Only for the cover letter, and not shortlog in general, I assume? It should be easy to set the struct shortlog fields to whatever suits your taste. Making it configurable (without getting tangled in other option parsing) is trickier, though. -Daniel *This .sig left intentionally blank*