Skip Menu |

This queue is for tickets about the activitymail CPAN distribution.

Report information
The Basics
Id: 13552
Status: resolved
Priority: 0/
Queue: activitymail

People
Owner: Nobody in particular
Requestors: chadh [...] pobox.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.21
Fixed in: 1.22



Subject: Format string changes with newer CVS (1.12.6+)
activitymail breaks with recent versions (1.12.6+) of CVS that have UseNewInfoFmtStrings set to 'yes' in the CVSROOT config file -- the default for newer installations (including Debian sarge, for example). In brief, the way %{sVv} is now expanded in the commitinfo file seems to make activitymail dump a usage message and bail. For an overview of the changes, see: https://www.cvshome.org/docs/manual/cvs-1.12.12/cvs_18.html#SEC188 A simple work-around is to add a '1' (numeral one) after the percent sign in the -f argument, enabling the old-style expansion. Documentation patch attached. Regards, Chad
--- activitymail.orig 2005-02-10 22:44:13.000000000 -0330 +++ activitymail 2005-07-06 12:21:28.000000000 -0230 @@ -1313,6 +1313,14 @@ actually have their diff header file names with underscores substituted for spaces. +Recent versions of CVS (1.12.6 and newer) use a different style of format +strings if C<UseNewInfoFmtStrings> is set to 'yes' in the CVSROOT F<config> +file (which may be the default for some installations). In that case, you +should write C<%1{sVv}> for activitymail's -f argument (i.e., add a numeral +one after the percent sign) to enable the old-style format strings it expects. +Note that you may receive deprecation warnings from CVS on check-in when using +this method. + =head1 TO DO =over 4
From: remi.ricard [...] simlog.com
Hi, Show quoted text
> activitymail breaks with recent versions (1.12.6+) of CVS that have > UseNewInfoFmtStrings set to 'yes' in the CVSROOT config file -- the > default for newer installations (including Debian sarge, for > example). In brief, the way %{sVv} is now expanded in the > commitinfo file seems to make activitymail dump a usage message and > bail. > > For an overview of the changes, see: > https://www.cvshome.org/docs/manual/cvs-1.12.12/cvs_18.html#SEC188 > > A simple work-around is to add a '1' (numeral one) after the percent > sign in the -f argument, enabling the old-style expansion. > Documentation patch attached. >
Since the work-around gives a warning to upgrade the script to use the new way. I was wondering if there is any development ? Remi
Fixed in 1.22.