Re: help/version patches

[Christopher Faylor <cgf at redhat dot com>]

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