<font size="2"><font face="tahoma,sans-serif">Hi;<br></font></font><br><div class="gmail_quote">On Fri, Jul 1, 2011 at 2:44 PM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr">baldrick@free.fr</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi Yuri,<br>
<br>
> +/*===-- Version -----------------------------------------------------------===*/<br>
> +<br>
> +typedef enum {<br>
> +  /* Terminator Instructions */<br>
<br>
^ Strange comment<br>
<br>
> +  LLVMVersionNumber     = 1,<br>
> +  LLVMVersionRepository = 2<br>
> +} LLVMVersion;<br>
> +<br>
> +const char* LLVMGetVersion(LLVMVersion VN);<br>
<br>
I think it would be better to get rid of the enum and have two functions.<br>
<br>
> +SVNVERSION   := $(shell cd $(PROJ_SRC_ROOT)&&  (LC_ALL=C svnversion -cn . 2>/dev/null | sed -e "s/.*://" -e "s/\([0-9]*\).*/\1/" | grep "[0-9]" || LC_ALL=C svn info . 2>/dev/null | awk '/^Revision:/ {print $$2 }' | grep "[0-9]" || LC_ALL=C git svn info . 2>/dev/null | awk '/^Revision:/ {print $$2 }' | grep "[0-9]" || echo unknown))<br>


> +CPPFLAGS     += -DREPOSITORY_REVISION="\"$(SVNVERSION)\""<br>
<br>
I don't much like this.  Probably there should be some helper for determining<br>
the revision.  And does everyone have sed and awk?  And what about when building<br>
a release so there is no repository, no svn or git?<br></blockquote><div><br>Here is the version.sh script from mplayer;<br><br># releases extract the version number from the VERSION file<br>version=$(cat VERSION 2> /dev/null)<br>

<br>if test -z $version ; then<br># Extract revision number from file used by daily tarball snapshots<br># or from the places different Subversion versions have it.<br>svn_revision=$(cat snapshot_version 2> /dev/null)<br>

test $svn_revision || svn_revision=$(LC_ALL=C svn info 2> /dev/null | grep Revision | cut -d' ' -f2)<br>test $svn_revision || svn_revision=$(grep revision .svn/entries 2>/dev/null | cut -d '"' -f2)<br>

test $svn_revision || svn_revision=$(sed -n -e '/^dir$/{n;p;q;}' .svn/entries 2>/dev/null)<br>test $svn_revision && svn_revision=SVN-r$svn_revision<br>test $svn_revision || svn_revision=UNKNOWN<br>version=$svn_revision<br>

fi<br><br>Regards,<br>ismail<br><br> <br></div></div>