Re: [gentoo-dev] demanual update (was: Don't use / when applying sed with CFLAGS)

[Mike Frysinger]

On Tuesday, June 28, 2011 08:25:50 Peter Volkov wrote:
> +Sometimes a package will not use the user's <c>${CFLAGS}</c> or
> +<c>${LDFLAGS}</c>.

there are more flag vars than this.  you should use language like:
Sometimes a package will not use the user's build settings (such as CFLAGS or 
LDFLAGS).

> -    sed -i -e "s:cc -O2:$(tc-getCC) ${CFLAGS} ${LDFLAGS}:" build.sh \
> +    sed -i -e "s|cc -O2|$(tc-getCC) ${CFLAGS} ${LDFLAGS}|" build.sh \

probably better to suggest delaying of the expansion in the first place.
sed -i -e "s|cc -O2|$(tc-getCC) \${CFLAGS} \${LDFLAGS}|" build.sh \

also, your example ignores CPPFLAGS
-mike