Re: [PATCH] add 'svn version' as 'svnversion' v1

>> I find it irritating to have a separate command to check for the
>> revision heterogeneity of a local working copy. The rationnal for
>> 'svnversion' as given in the source file is that it could have been
>> used in the svn building process. Well, it is not, and svnversion does
>> still exists anyway after this patch.
>
> It's used in 'make install':
>
> $ cd ~/src/subversion
> $ sudo make install
> Password: **********
> [...]
> subversion/svnversion/svnversion . /repos/svn/trunk \
> > /usr/local/include/subversion-1/svn-revision.txt
> $
>
> However, that could just as easily be the command-line client as the
> svnversion program. It's invoked from the just-built tree, after all.

Yes, I saw this one while testing my patch;-)
It is the very last thing performed by the "make install".
As you point out, "svn something" would also do the job.

The comment in svnversion/main.c suggested that it could have be used
during the compilation phase, so as to produce a string embedded within
the "svn" executable itself. As there would be no opportunity to execute
"svn something" before "svn" is linked, there was a potential need for
light svn executable with just this functionnality.

Anyway, I'll keep in mind this use. Thanks for your point.

-- 
Fabien.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org