[FFmpeg-devel] [PATCH] Remove old VCSs from version.sh
Mon Feb 21 11:33:24 CET 2011
On Mon, Feb 21, 2011 at 2:17 AM, Alex Converse <alex.converse at gmail.com> wrote:
> On Mon, Feb 21, 2011 at 2:09 AM, Reinhard Tartler <siretart at tauware.de> wrote:
>> On Mon, Feb 21, 2011 at 11:05:24 (CET), Alex Converse wrote:
>>> Can you give an example of where this snapshot_version is currently used?
>> In 'my' packaging for debian.
>>> The original documentation for snapshot_version says:
>>> Read revision string from the file snapshot_version if available.
>>> This will be used by daily snapshots without Subversion metadata.
>>> It is in the group of revision probes that get prefixed with SVN-r
>>> which implies that if it is still used it is at best wrong.
>> Seems largely correct, but of course needs to be reworded to be
>> SVN-agnostic. It is basically needed to override foreign, bad, or
>> missing VCS information.
> Don't we already have a VERSION file for when there is no VCS info?
On further inspection, I don't care. Attached is an updated patch;
snapshot_version remains as-is.
Anyone who wants to come up with a modern interpretation or
replacement of this snapshot_version file is welcome to do so after
this has landed. After all, none of the snapshot_version defenders
seemed to care about it making sense until I tried to remove it.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1662 bytes
Desc: not available
More information about the ffmpeg-devel