[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] commit-email.pl to produce mbox messages to stdout

From: Lars Müller <lm_at_openslx.com>
Date: 2006-12-06 20:24:33 CET

Hi,

On Wed, Dec 06, 2006 at 10:40:33AM -0500, C. Michael Pilato wrote:
> So, I like the idea of this patch. And the implementation seems fine,
> as far as my testing and review can tell. Committed in r22589.

Thanks a lot.

> I would recommend now that we add a command-line option for forcing
> stdout mode. And then I would recommend one of two followup actions:
>
> * Either deprecate the -l option now that folks can use
> commit-email.pl --stdout >> logfile (and get mbox format!)
>
> * Or at least make the -l option add the mbox From header.

Attached as diff a version based on rev 22589 with a --stdout command
line option. I reverted all former and not superfluous changes made
with rev 22589.

Your second suggestion is already the case as the -l option uses the
same @head as stdout does.

By intention I did not remove the -l option. This is up to one of the
subversion committers.

Might some users use -l to pass the message to a file and afterwards
perform modifications on this file? If this is the case it might be
usefull to stay with -l further. Yes, these users might even pipe the
output to their modifying commands or redirect the output to a file.
The only potential problem I see is after an update to a newer
subversion version. A missing -l option might then break existing
scripts.

While testing I ran into the problem of a non valid mbox output as soon
as the locale is set to something non US or non POSIX. The problem is
the first line of the output. I first thought about circumventing this
by setting the locale to POSIX while output the particular line. But we
might consider to document this problem instead.

Lars

  • application/pgp-signature attachment: stored
Received on Wed Dec 6 20:25:40 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.