From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-oa0-x231.google.com ([2607:f8b0:4003:c02::231]) by metis.ext.pengutronix.de with esmtp (Exim 4.72) (envelope-from ) id 1XC8YW-0002xK-Rx for ptxdist@pengutronix.de; Tue, 29 Jul 2014 16:39:10 +0200 Received: by mail-oa0-f49.google.com with SMTP id eb12so10282045oac.36 for ; Tue, 29 Jul 2014 07:39:02 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <53D79F97.9060804@pengutronix.de> References: <53D79F97.9060804@pengutronix.de> Date: Tue, 29 Jul 2014 16:39:02 +0200 Message-ID: From: Kristof Verdonck Subject: Re: [ptxdist] configure failed with sed version 4.2.2 Reply-To: ptxdist@pengutronix.de List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============1608314601==" Sender: ptxdist-bounces@pengutronix.de Errors-To: ptxdist-bounces@pengutronix.de To: ptxdist --===============1608314601== Content-Type: multipart/alternative; boundary=047d7bd6ac482d7ecc04ff560083 --047d7bd6ac482d7ecc04ff560083 Content-Type: text/plain; charset=UTF-8 I'm using an ancient ptxdist, but that doesn't matter because I'm isolating the SED-version check. If I run the regex from http://git.pengutronix.de/?p=ptxdist.git;a=blob;f=configure.ac;h=22c8df03a8c225514a3df7ba4c9fb29c515c3e24;hb=HEAD sed --version 2>/dev/null | sed -ne "1 s/.*GNU.*\s\([[0-9\+\.]]\+\).*/\1/p" then it returns empty, both with sed 4.2.1 and 4.2.2. 2014-07-29 15:20 GMT+02:00 Marc Kleine-Budde : > Hello Kristof, > > thanks for your mail, please use the ptxdist mailinglist (Cc'ed) in the > future. > > On 07/29/2014 03:08 PM, Kristof Verdonck wrote: > > I had a problem with the ptxdist configure script for checking the SED > > version. > > Which version of ptxdist are you using? > > > Apparently this is a known issue that hasn't been fixed yet. > > https://www.mail-archive.com/ptxdist@pengutronix.de/msg06717.html > > I think the problem has been fixed with the patch: > > 2a89985 configure.ac: change regex for sed version > > > I took the regular expression GLIBC used to check the SED version. > > I tested it with sed 3.02.80, 4.1.5, 4.2.1 and 4.2.2 > > I thought you could use this. > > > > SED_VERSION=`$SED --version 2>/dev/null | $SED -n "s/^.*GNU sed[^0-9]* > > \([0-9]*\.[0-9.]*\).*$/\1/p"` > > AFAIK we don't have any problems with sed since the above patch in > mainline, which is ptxdist-2013.01.0. > > Marc > > -- > Pengutronix e.K. | Marc Kleine-Budde | > Industrial Linux Solutions | Phone: +49-231-2826-924 | > Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | > > --047d7bd6ac482d7ecc04ff560083 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
I'm using an ancient ptxdis= t, but that doesn't matter because I'm isolating the SED-version ch= eck.

If I run the regex from=C2=A0http://git.pengutronix.de/?p=3Dptxdist.git;a=3Db= lob;f=3Dconfigure.ac;h=3D22c8df03a8c225514a3df7ba4c9fb29c515c3e24;hb=3DHEAD=
sed=C2=A0--version=C2=A02>/dev/null=C2=A0|=C2=A0sed=C2=A0-ne= =C2=A0"1=C2=A0s/.*GNU.*\s\([[0-9\+\.]]\+\).*/\1/p" then it returns empty, both with sed 4.2.1 and 4.2.2.


2014-07-29 15:20 GMT+02:00 Marc Kleine-Budde <mkl= @pengutronix.de>:
Hello Kristof,

thanks for your mail, please use the ptxdist mailinglist (Cc'ed) in the=
future.

On 07/29/2014 03:08 PM, Kristof Verdonck wrote:
> I had a problem with the ptxdist configure script for checking the SED=
> version.

Which version of ptxdist are you using?

> Apparently this is a known issue that hasn't been fixed yet.
> https://www.mail-archive.com/ptxdist@pengutronix.= de/msg06717.html

I think the problem has been fixed with the patch:

=C2=A0 =C2=A0 2a89985 con= figure.ac: change regex for sed version

> I took the regular expression GLIBC used to check the SED version.
> I tested it with sed 3.02.80, 4.1.5, 4.2.1 and 4.2.2
> I thought you could use this.
>
> SED_VERSION=3D`$SED --version 2>/dev/null | $SED -n "s/^.*GNU = sed[^0-9]*
> \([0-9]*\.[0-9.]*\).*$/\1/p"`

AFAIK we don't have any problems with sed since the above patch i= n
mainline, which is ptxdist-2013.01.0.

Marc

--
Pengutronix e.K. =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0| Marc Kleine-Budde =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |
Industrial Linux Solutions =C2=A0 =C2=A0 =C2=A0 =C2=A0| Phone: +49-231-2826-924 =C2=A0 = =C2=A0 |
Vertretung West/Dortmund =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| Fax: =C2=A0 +49-5121-2= 06917-5555 |
Amtsgericht Hildesheim, HRA 2686 =C2=A0| http://www.pengutronix.de =C2=A0 |



--047d7bd6ac482d7ecc04ff560083-- --===============1608314601== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline -- ptxdist mailing list ptxdist@pengutronix.de --===============1608314601==--