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