On Sun, May 31, 2009 at 06:45:28PM +0200, uzsolt wrote:
> Author: uzsolt Date: Sun May 31 16:45:28 2009 GMT
> Module: packages Tag: HEAD
> ---- Log message:
> - initial
> +%prep
> +%setup -q
> +sed -i "s...@^prefix.*@PREFIX=%{_prefix}@" config.mk
> +sed -i "s...@^\(CFLAGS.*\)-Os\(.*\)@\1 \2 %{rpmcflags}@" config.mk
> +sed -i "s...@^\(LDFLAGS.*\)@\1 %{rpmldflags}@" config.mk
You shouldn't be defining these at %prep stage, doing so in %build is
much more secure. And there must be much a simpler way than using sed
to do it. You could pass options in %__make invocation, or append them
at the end of config.mk file:
%build
cat << 'EOF' >> config.mk
PREFIX = %{_prefix}
CFLAGS := %{rpmcflags} $(filter-out -Os,$(CFLAGS))
LDFLAGS = %{rpmldflags}
EOF
%{__make}
BTW, appropriate sed BR is missing.
--
____ Sparky{PI] -- Przemyslaw _ ___ _ _ ........... LANG...Pl..Ca..Es..En
/____) ___ ___ _ _ || Iskra | | _ \| | | : WWW........ppcrcd.pld-linux.org
\____\| -_)'___| ||^'||//\\// < | _/| | | : JID......sparky<at>jabberes.org
(____/|| (_-_|_|| ||\\ || |_ |_| |_| _| : Mail....sparky<at>pld-linux.org
_______________________________________________
pld-devel-en mailing list
[email protected]
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
