[coreboot] [LinuxBIOS] [PATCH] superiotool: Make --version actually work correctly
todthgie at hotmail.com
Sun Oct 19 11:15:42 CEST 2008
im i the progress of parsing my old email and have a question:
- why not output something like "superiotool r2828 (2814:2828M)" or
"superiotool r2828 (details:2814:2828M)"
Ulf Jordan wrote:
> On Mon, 8 Oct 2007, Uwe Hermann wrote:
>> On Sun, Oct 07, 2007 at 11:09:24AM +0200, Ulf Jordan wrote:
>>> Attached is a patch that:
>>> * derives the superiotool version number from the latest committed
>>> to superiotool files (NOT the latest svn up of the superiotool
>> Hm, isn't that wrong? We want to know the extact version from which the
>> binary was built, _not_ the most current version in the repository at the
>> time it was built...
> My formulation was inexact. svnversion shows the local revisions of the
> working copy, not the repository. What I meant to say was:
> * derives the superiotool version number from the highest revision
> number of the checked out superiotool files (NOT the revision number
> from latest svn up of the superiotool directory)
>>> * strips away extra information from svnversion, to arrive at a single
>>> version number like 2828 instead of 2814:2828M ("this build is made
>>> up of
>>> files with latest modification in rev 2814 to 2828 plus local
>> Hm, maybe that would even be a good thing? We could tell from that
>> output that a user used a modified version and ask for a diff...
> Yes maybe. More information at the expense of a more odd looking version
> The stripping is implemented by the sed invocation, without which we
> would get the more detailed information. I'll leave it as is, awaiting
> further discussion on what we actually want to show in the version number.
>>> This approach could easily be combined with a traditional version number
>>> from a #define in superiotool.h, as Robinson pointed out.
>> Not needed and not useful. We don't want to confuse people with _two_
>> versioning systems.
>>> Index: Makefile
>>> --- Makefile (revision 2828)
>>> +++ Makefile (working copy)
>>> @@ -24,14 +24,19 @@
>>> INSTALL = /usr/bin/install
>>> PREFIX = /usr/local
>>> +SVNDEF := -D'SUPERIOTOOL_VERSION="$(shell svnversion -cn . \
>>> + | sed -e "s/.*://" -e "s/\([0-9]*\).*/\1/")"'
>> Please add a comment here which explains what the contents of the
>> variable will be (svn rev of latest commit etc...), how it works, etc.
> OK, added.
> Attached is an updated patch.
Express yourself instantly with MSN Messenger! Download today it's FREE!
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the coreboot