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

Re: [PATCH] add support for ignoring white-space to commit-email.pl

From: Greg Stein <gstein_at_gmail.com>
Date: Wed, 21 Jan 2009 12:23:10 +0100

Yeah, that script is deprecated. Maybe somebody can apply the patch
for you, but really... I'd suggest switching over to mailer.py. Maybe
this patch could be redone for that script instead?

Also note: you didn't attach the actual patch :-P

Cheers,
-g

On Wed, Jan 21, 2009 at 11:41, Vadim Zeitlin <vadim_at_wxwidgets.org> wrote:
> Hello,
>
> I'd like to propose a simple patch to commit-email.pl script which adds
> --ignore-all-space option to it which is simply passed to svnlook unchanged.
> I've just discovered -- after making the patch, of course -- that this
> script is now deprecated but maybe this patch could still be applied.
>
> The patch was not done using svn but is against the latest revision of
> contrib/hook-scripts/commit-email.pl.in the trunk (r34134). Please let me
> know if you see any problems with it.
>
> Thanks,
> VZ
>
> [[[
> Add --ignore-all-space option to commit-email.pl with the same meaning as
> the option with the same name in svnlook diff.
>
> Signed-off-by: Vadim Zeitlin <vadim_at_wxwidgets.org>
>
> commit-email.pl.in | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
> ]]]
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1040845
> Hello,
>
> I'd like to propose a simple patch to commit-email.pl script which adds
> --ignore-all-space option to it which is simply passed to svnlook unchanged.
> I've just discovered -- after making the patch, of course -- that this
> script is now deprecated but maybe this patch could still be applied.
>
> The patch was not done using svn but is against the latest revision of
> contrib/hook-scripts/commit-email.pl.in the trunk (r34134). Please let me
> know if you see any problems with it.
>
> Thanks,
> VZ
>
> [[[
> Add --ignore-all-space option to commit-email.pl with the same meaning as
> the option with the same name in svnlook diff.
>
> Signed-off-by: Vadim Zeitlin <vadim_at_wxwidgets.org>
>
> commit-email.pl.in | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
> ]]]
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1040845
>

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1040890
Received on 2009-01-21 12:23:27 CET

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.