This is the mail archive of the
cygwin-patches@cygwin.com
mailing list for the Cygwin project.
Re: help/version patches
On Tue, Feb 26, 2002 at 08:59:45PM -0800, Joshua Daniel Franklin wrote:
>Well, except that that loop does away with the whole point--eating the
>trailing '$'. Here's another go. I tested it with "$Revision$",
>"$Revision:$", "$Revision: $ ", "$Revision: 1.22 $ ", and "$Revision: 1.2 2 $ "
>(in the unlikely event of a version number with spaces) and it does the right
>thing on each.
There's no need for a loop. There really shouldn't be any need to
accommodate the missing colon but it doesn't hurt too much to add a test
case.
I've checked in a modified version of your patch. I cleaned up some of
the non-GNU formatting, added a ChangeLog, and added a "print_version"
function which parses the 'version' array for version info.
I appreciate the submission very much but, next time, I would also
appreciate a ChangeLog and more attention to formatting issues.
cgf