id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc
125	Simplify sed's	gerard@…	markh@…	"Proposed by Handy Hron:

In chapter 5 bzip2-1.0.1 is currently:
sed \
   s/""\$(CC) \$(CFLAGS) -o""/""\$(CC) \$(CFLAGS) \$(LDFLAGS) -o""/ \
 
proposed:
sed  \
   's/$(CC) $(CFLAGS) -o/$(CC) $(CFLAGS) $(LDFLAGS) -o/' \
 
The other one in chapter 5 is Mawk.  It uses the same sed as bzip2.
 
In Chapter 6 there are a couple seds would be simpler if they used the
feature that:  's///' == 's|||'   There is already a sed using this feature
in chapter 6  (Util-linux-2.11h)

glibc-2.2.3
currently:
   sed s/""\$(PERL)""/""\/usr\/bin\/perl""/ \
   ...
   sed s/""cross-compiling = yes""/""cross-compiling = no""/ \
 
proposed:
   sed 's|$(PERL)|/usr/bin/perl|'  \
   ...
   sed 's/cross-compiling = yes/cross-compiling = no/ \
 
Procinfo-18 has double quotes.  They could be switched to singles for
consistancy.
 
Util-linux-2.11h
currently:
   sed ""s|etc/adjtime\""\$|var/lib/hwclock/adjtime\""|"" \
 
proposed:
   sed 's|etc/adjtime|var/lib/hwclock/adjtime|' \"	defect	closed	normal		Book	CVS	normal	fixed