Gentoo Archives: gentoo-dev

From: Samuli Suominen <ssuominen@g.o>
To: gentoo-dev@l.g.o, xmw@g.o
Subject: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in media-libs/libexif: ChangeLog libexif-0.6.20.ebuild
Date: Sun, 13 Nov 2011 18:21:00
Message-Id: 4EC00A15.6060309@gentoo.org
1 On 11/13/2011 08:07 PM, Michael Weber (xmw) wrote:
2 > xmw 11/11/13 18:07:16
3 >
4 > Modified: ChangeLog libexif-0.6.20.ebuild
5 > Log:
6 > remove -g from FLAGS, as pointed out by Agostino Sarubbo (bug 390249)
7 > @@ -22,7 +22,11 @@
8 >
9 > src_prepare() {
10 > epatch "${FILESDIR}"/${PN}-0.6.13-pkgconfig.patch
11 > - elibtoolize # FreeBSD .so version
12 > + #elibtoolize # FreeBSD .so version
13 > + sed -e '/FLAGS=/s:-g::g' \
14 > + -e '/FLAGS=/s:-Wall::g' \
15 > + -i configure.ac
16 > + eautoreconf
17 > }
18
19 removing compile warning flags is not allowed, thus hiding also
20 Portage's QA messages. restore -Wall.
21
22 and running eautoreconf just to remove -g from configure is too heavy.
23 just sed the configure directly.
24
25 >
26 > src_configure() {
27 > @@ -38,5 +42,4 @@
28 > emake DESTDIR="${D}" install || die
29 > find "${D}" -name '*.la' -exec rm -f {} +
30 > rm -f "${D}"usr/share/doc/${PF}/{ABOUT-NLS,COPYING}
31 > - prepalldocs
32 > }
33
34 the prepalldocs was still used because the ebuild is not EAPI=4. restore
35 it or upgrade the EAPI.