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

[Alex Converse]

On Mon, Feb 21, 2011 at 2:33 AM, Alex Converse <alex.converse at gmail.com> wrote:
>
> 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.

This version pushed.