[LinuxBIOS] r2815 - trunk/util/superiotool
Uwe Hermann
uwe at hermann-uwe.de
Mon Oct 1 22:43:24 CEST 2007
On Mon, Oct 01, 2007 at 03:52:22PM +0200, Stefan Reinauer wrote:
> svn at openbios.org wrote:
> > De-uglify the --version output (trivial).
> >
>
> > - printf("superiotool %s\n", SUPERIOTOOL_VERSION);
> > + strncpy((char *)&tmp,
> > + (const char *)&SUPERIOTOOL_VERSION[6],
> > + strlen(SUPERIOTOOL_VERSION) - 8);
> > + printf("superiotool r%s\n", (char *)&tmp);
> >
>
> Hm.. is this less uglier? Does it look trivial? ;)
OK, probably not trivial :) Guilty as charged.
It's less uglier (in the output), though:
Before:
./superiotool -v
superiotool r$Rev: 2814 $
After:
./superiotool -v
superiotool r2815
> Is there any reason that this doesnt just say
>
> printf ("superiotool r" SUPERIOTOOL_VERSION "\n");
Yes, it won't work unfortunately (I hoped it would, but it doesn't).
$Rev$ will not be replaced by the svn revision (e.g. "2815")
but rather by "$Rev: 2815 $" (which is stupid IMO, but we cannot change
that). It would be nice if there was something like $OnlyRev$ or so
which only prints the actual number, but I don't think there's such an
option.
Uwe.
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20071001/a95f7fa8/attachment.sig>
More information about the coreboot
mailing list