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

[Alex Converse]

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...
Name: 0001-Remove-old-VCSs-from-version.sh.patch
Type: text/x-patch
Size: 1662 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110221/718a99dc/attachment.bin>