Gentoo Archives: gentoo-dev

From: Donnie Berkholz <dberkholz@g.o>
To: gentoo-dev@l.g.o
Cc: ribosome@g.o
Subject: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in sci-biology/phylip: ChangeLog phylip-3.67.ebuild
Date: Thu, 20 Sep 2007 01:45:34
Message-Id: 20070920013306.GF31094@supernova
1 On 18:32 Wed 19 Sep , Olivier Fisette (ribosome) wrote:
2 > Modified: ChangeLog
3 > Added: phylip-3.67.ebuild
4 > Log:
5 > New upstream version.
6 > (Portage version: 2.1.3.9)
7 >
8
9 > file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/sci-biology/phylip/ChangeLog?rev=1.27&view=markup
10 > plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/sci-biology/phylip/ChangeLog?rev=1.27&content-type=text/plain
11 > diff : http://sources.gentoo.org/viewcvs.py/gentoo-x86/sci-biology/phylip/ChangeLog?r1=1.26&r2=1.27
12
13 > src_compile() {
14 > sed -e "s/CFLAGS = -O3 -fomit-frame-pointer/CFLAGS = ${CFLAGS}/" \
15 > -e "s/CC = cc/CC = $(tc-getCC)/" \
16 > -e "s/DC = cc/DC = $(tc-getCC)/" \
17 > -i Makefile || die "Patching Makefile failed."
18
19 This sed is pretty fragile, since it relies on upstream not changing the
20 spacing or the default CFLAGS. Better tokens might do something like this:
21
22 sed -e "s:^\(CFLAGS.*=\).*:\1 ${CFLAGS}:g"
23
24 Thanks,
25 Donnie
26 --
27 gentoo-dev@g.o mailing list

Replies