From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pigeon.gentoo.org ([208.92.234.80] helo=lists.gentoo.org) by finch.gentoo.org with esmtp (Exim 4.60) (envelope-from ) id 1RcESt-0004Ws-Al for garchives@archives.gentoo.org; Sun, 18 Dec 2011 10:59:35 +0000 Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id A2FC721C187; Sun, 18 Dec 2011 10:59:24 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) by pigeon.gentoo.org (Postfix) with ESMTP id 83D3821C0BA for ; Sun, 18 Dec 2011 10:58:48 +0000 (UTC) Received: from pomiocik.lan (77-253-147-68.adsl.inetia.pl [77.253.147.68]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: mgorny) by smtp.gentoo.org (Postfix) with ESMTPSA id E679E1B400A; Sun, 18 Dec 2011 10:58:46 +0000 (UTC) Date: Sun, 18 Dec 2011 11:59:30 +0100 From: =?UTF-8?B?TWljaGHFgiBHw7Nybnk=?= To: gentoo-dev@lists.gentoo.org Cc: jlec@gentoo.org Subject: Re: [gentoo-dev] checking C*FLAGS used for build (was: making the stable tree more up-to-date) Message-ID: <20111218115930.035d7ea8@pomiocik.lan> In-Reply-To: <4EEB2F8F.2000504@gentoo.org> References: <4ECA0EA3.8020407@gentoo.org> <1769026.9R6ACMvGeL@devil> <4EEB2715.5060806@gentoo.org> <2952499.vjcSCuKqgc@devil> <4EEB2F8F.2000504@gentoo.org> Organization: Gentoo X-Mailer: Claws Mail 3.7.10 (GTK+ 2.24.8; x86_64-pc-linux-gnu) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=PGP-SHA256; boundary="Sig_/YDzaPXBFaHFx/ECVaSOAN9N"; protocol="application/pgp-signature" X-Archives-Salt: e5627061-70c1-42c3-9bd2-7f7f4f496fb4 X-Archives-Hash: 5b3d1f0546eb2226fad6eb461a0fe217 --Sig_/YDzaPXBFaHFx/ECVaSOAN9N Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Fri, 16 Dec 2011 12:46:23 +0100 justin wrote: > On 12/16/11 12:21 PM, Agostino Sarubbo wrote: > > On Friday 16 December 2011 06:10:13 Anthony G. Basile wrote: > >> Does your script do any checking on the quality of the ebuild, eg > >> that it respects C/LDFLAGS. If so, that's useful and would help > >> package maintainers to better prepare their ebuilds for > >> stabilization. > > Unfortunately no.=20 > >=20 > > For LDFLAGS there is a QA warning and is enough visible > > For CFLAGS I see with the naked eye a bit of build log > > My script at "end of work" just runs repoman full and cat entire > > ebuild( so, imho, should be a tasks already done by maintainers). > > Finally, I take a look at the ebuild to see if there are issue(s) > >=20 > > This is all. > >=20 > >> And congrats on making dev > > Thanks ;) > >=20 > >=20 > > Regards > > Agostino >=20 > You can use the post* hooks for that. For FLAGS mixing I am using > CFLAGS and CXXFLAGS specific warning flags during compilation and > grep for the gcc mixing warning in the build log. Probably similar > things can be done for other problematic points. There was also a > more sophisticated approach Diego blogged about. >=20 > For respecting C/CXXFLAGS there was something Donnie suggested long > ago. If you use -frecord-gcc-switches you are able to read the used > LFGAS with eu-readelf. That sounds neat. Maybe we should make portage optionally support it for now (i.e. when user uses such a flag him-/herself)? Sadly, I don't see a neat way of splitting those switches between separate files being built. But better that than nothing, I guess. --=20 Best regards, Micha=C5=82 G=C3=B3rny --Sig_/YDzaPXBFaHFx/ECVaSOAN9N Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (GNU/Linux) iJwEAQEIAAYFAk7tx5kACgkQfXuS5UK5QB0+RAQAjXWbIF6+ihA5crqen6fcJtoW 3QI5ThCnk5FDNBgqf9B8/75VgG21hAimxa8Ko04nVHVG6RmdhNv/SDe3wzH2R2Kr M4Pe5a24QWB+sE1SI/SiNr5lZItXqydknWGH7k+rqgCd2A4OqO90SJuVTPQMutU8 pVzUnxhY/uZdJKd1yk8= =iRM0 -----END PGP SIGNATURE----- --Sig_/YDzaPXBFaHFx/ECVaSOAN9N--