[FFmpeg-devel] [PATCH] Remove old VCSs from version.sh

Måns Rullgård mans
Mon Feb 21 12:09:15 CET 2011


Alex Converse <alex.converse at gmail.com> writes:

> 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?

Yes, and it's used verbatim, so there are no issues with changing VCS.
Whatever scripts use snapshot_version should simply be updated to use
VERSION instead.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list