Re: pkg/47282: security/otptool: portability issue with sed (sed -r)

["OBATA Akio" <obache%netbsd.org@localhost>]

The following reply was made to PR pkg/47282; it has been noted by GNATS.

From: "OBATA Akio" <obache%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/47282: security/otptool: portability issue with sed (sed -r)
Date: Sun, 30 Dec 2012 16:29:34 +0900

 On Tue, 04 Dec 2012 17:20:02 +0900, <draco%marino.st@localhost> wrote:
 
 >> Description:
 > otptool uses the gnu option "-r" which is analogous to "-E" on BSD.  If sed 
 > doesn't support -r (as seen with DragonFly), then otptool is fails to build:
 >
 > --- module ---
 > if test "." != "."; then /bin/cp ./mod_authn_otp.c .; fi
 > /usr/pkg/sbin/apxs -c -D_REENTRANT `echo -Wall -Waggregate-return 
 > -Wcast-align -Wchar-subscripts -Wcomment -Wformat         -Wimplicit 
 > -Wmissing-declarations -Wmissing-prototypes -Wnested-externs -Wno-long-long  
 >        -Wparentheses -Wpointer-arith -Wredundant-decls -Wreturn-type 
 > -Wswitch -Wtrigraphs         -Wuninitialized -Wunused -Wwrite-strings 
 > -Wshadow -Wstrict-prototypes -Wcast-qual | sed -r 's/ -/ -Wc,-/g'` -l crypto 
 > mod_authn_otp.c
 > sed: illegal option -- r
 > usage: sed script [-Ealn] [-i extension] [file ...]
 >        sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file 
 > ...]
 
 Using 's/ -/ -Wc,-/g', which is not a extended regular expressions,
 so it is safe to drop `-r' option.
 (already fixed by upstream, v1.1.5: 
https://code.google.com/p/mod-authn-otp/source/detail?spec=svn129&r=116 )
 
 -- 
 OBATA Akio / obache%NetBSD.org@localhost