[Mplayer-cvslog] CVS: main configure,1.653,1.654

[D Richard Felker III]

On Thu, Feb 13, 2003 at 01:54:30PM -0500, D Richard Felker III wrote:
> On Thu, Feb 13, 2003 at 06:15:42PM +0100, Felix Buenemann wrote:
> > On Wednesday 12 February 2003 23:44, Arpi wrote:
> > > Hi,
> > >
> > > > > sed -e 's/^\([0-9]*\)\.\([0-9]*\).*/\1\2/'` _def_faad_version="#define
> > > > > FAADVERSION $_faad_tempversion"
> > > > >      echores "yes ($_faad_version)"
> > > > >    else
> > > >
> > > > ??? why you changed it, before it could match 1.0, 10.2, 10.23 etc, now
> > > > it will also match "." which is wrong (* means none or more, + means one
> > > > or more).
> > >
> > > that version with + didn't work. later someone at -users told me that
> > > + has to be escaped (so \+ ) in sed... feel free to fix.
> > but it is \+ in the original code, ...
> 
> Somebody didn't RTFM!! sed does not use extended or perl regex's, just
> basic ones, which do not have any form of +. If you want that
> functionality, put the pattern twice and follow the second instance of
> it by a *. I still think this is a nonsensical way to figure out a
> version number. See my post in the thread a while back for a correct
> way.

Hmm, not in this thread I guess, maybe an earlier one here or on
-dev-eng. The basic idea is just to use sed to replace the . by
"*100+" or "*1000+" then use $[`echo ... | sed ...`] to make the shell
evaluate it.

Rich