[MPlayer-dev-eng] [PATCH] proper version info for git.mplayerhq.hu

Rudolf Polzer divVerent at alientrap.org
Tue Jul 27 06:58:37 CEST 2010


On Mon, Jul 26, 2010 at 10:45:55PM +1000, Rowan James wrote:
> 
> On 26/07/2010, at 7:40 PM, Rudolf Polzer wrote:
> 
> > On Mon, Jul 26, 2010 at 09:05:31AM +0000, Carl Eugen Hoyos wrote:
> >> Rudolf Polzer <divVerent <at> alientrap.org> writes:
> >> 
> >>> I attached a small patch that, if mplayer is compiled from mplayer.git, uses
> >>> the git commit hash instead of UNKNOWN as version info.
> >> 
> >> Wouldn't it be better to show the svn version from which the git repository was
> >> cloned? (I believe this is how FFmpeg does it.)
> > 
> > Only as long as the git stays read-only. Just because no such move is planned
> > YET, it does not mean it won't happen in some years.
> > 
> > I attached a more elaborate version. It now shows, "in order of likelihood":
> > <...>
> 
> 
> Where there is an ID for a commit in 'master' included, do you mean 'origin/master'?

Yes, see patch.

> The building user's 'master' branch could potentially include any number of local commits never to be pushed - in fact, it's the default HEAD for such commits to be added to; and you really don't want to squelch the origin commit it was based on.  I know I do all sorts of weird and wonderful things with my local master for my 'day job' between pushes.

Exactly.

> I like the showing of SVN revision where it's available; but once there are git commits involved, you want to make sure you indicate which (origin/master) commit is most recent in the current branch, and as you mention, _if_ there are any others.
> 
> So, to fit my pattern-substitution oriented thought process:
> 
> (official)(+local tracked)(+LOCAL)-4.4.1
> 
> Where:
> 'official' would be either the SVN r.#, or the origin/master commit ID
> 'local tracked' would be the latest git commit ID on the current branch, if different from 'official' - nothing for SVN-sourced working copies.
> '+LOCAL' indicates untracked changes, as before

That is what it does, yes.

> Even this doesn't sit well with users cloning from each other (interesting mental image...); as origin/master will point to whatever remote branch they started with.  The effect of that should be limited though, the ID will just not be any commit in the official repo...

But as long as the official repo is git-svn based, it will be an obvious difference.


More information about the MPlayer-dev-eng mailing list