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

Rudolf Polzer divVerent at alientrap.org
Fri Dec 17 09:24:25 CET 2010


On Thu, Dec 16, 2010 at 01:33:05PM +0100, Diego Biurrun wrote:
> On Wed, Dec 15, 2010 at 07:44:29PM +0100, Rudolf Polzer wrote:
> > On Wed, Dec 15, 2010 at 04:51:50PM +0100, Diego Biurrun wrote:
> > > On Wed, Dec 15, 2010 at 03:18:29PM +0100, Clément Bœsch wrote:
> > > > On Wed, Dec 15, 2010 at 03:14:43PM +0100, Diego Biurrun wrote:
> > > > > On Thu, Dec 09, 2010 at 07:30:59AM +0100, Rudolf Polzer wrote:
> > > > > > 
> > > > > > Then the attached one should be better.
> > > > > 
> > > > > Use 4 spaces for indentation, put a space after '>' and replace `` with $().
> > > > 
> > > > Why $() is prefered over ``?
> > > 
> > > $() is no less portable in practice and saner to nest.
> > 
> > I do remember once having used a system where `` worked and $() did not. But I
> > doubt these systems run mplayer anyway.
> 
> Probably some old Solaris crap default userland, but such a system won't
> even run MPlayer's configure...

Fine, attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: git-revision-in-version-sh.diff
Type: text/x-diff
Size: 1861 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20101217/119d1fc9/attachment.diff>


More information about the MPlayer-dev-eng mailing list