[FFmpeg-devel] [PATCH] Read "Last Changed Rev" field as version info.

[Reimar Döffinger]

On Fri, Jan 15, 2010 at 11:41:31PM +0000, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> > On Fri, Jan 15, 2010 at 10:53:27PM +0000, M?ns Rullg?rd wrote:
> >> Ramiro Polla <ramiro.polla at gmail.com> writes:
> >> > Read "Last Changed Rev" field instead of "Revision" field to avoid
> >> > being affected by changes in other directories (such as branches).
> >> > This will be more useful for getting swscale revision that isn't
> >> > affected by other commits to mplayer.
> >> >
> >> > I tested on subversion 1.3 for the old .svn/entries format, but I left
> >> > out the new .svn/entries format because I'm not that skilled with sed
> >> > and I don't know how reliable it would be to get the nth line after
> >> > "dir".
> >> 
> >> Why do we look for the revision in so many ways?  Does "svn info"
> >> sometimes fail?
> >
> > When it is not installed.
> > Or the checkout was done from a different machine with a newer SVN version
> > (yes, unfortunately there are development systems around with such outdated
> > software).
> > Probably more cases if you think about it hard enough :-)
> 
> Well, if people feel the need, and it's only limited to a couple of
> lines...

Btw. I just remembered one of the bigger cases of "svn not installed", any
Windows or OSX user using a SVN GUI (those usually build on libsvn, not the svn
binary, so the later one is not installed, and for Windows such a Gui basically
is the only way to get something that isn't a pain to update without using cygwin).