With the optimizations I wanted to use for compiling on the Crusoe processor:
-O2 -fno-strict-aliasing -fno-common -fomit-frame-pointer -pipe
-mpreferred-stack-boundary=2 -march=i686 -malign-functions=0 -malign-jumps=0
-malign-loops=0
the sys-apps/hdparm ebuild fails because in src_unpack(), the sed command
replaces the -mpreferred-stack-boundary with '-mpreferredtack-boundary' instead
of removing the '-s' flag from the command line as was intended.
In general, any time sed is used to place the CFLAGS in a Makefile, we ought to
do any changes/deletions to the file before we insert the CFLAGS. In other
words, unless you really mean to potentially change the user's CFLAGS, the
CFLAGS insertion ought to come last in the sed command.
Here's a fix for hdparm.
--- hdparm-4.6.ebuild Sun Mar 10 07:53:31 2002
+++ hdparm-4.6-r1.ebuild Sun Mar 10 07:50:15 2002
@@ -12,7 +12,7 @@
unpack ${A}
cd ${S}
mv Makefile Makefile.orig
-
sed -e "s/-O2/${CFLAGS}/" -e "s:-s::" \
+
sed -e "s:-s::" -e "s/-O2/${CFLAGS}/" \
Makefile.orig > Makefile
}
