Attachment 320234 Details for Bug 429744 – common-mistakes.diff

[patch] common-mistakes.diff

common-mistakes.diff (text/plain), 1.58 KB, created by Julian Ospald on 2012-08-04 00:40:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Julian Ospald

Created: 2012-08-04 00:40:28 UTC

Size: 1.58 KB

patch

obsolete

>From: Julian Ospald <hasufell@gentoo.org>
>Date: Sat Aug  4 00:33:15 UTC 2012
>Subject: devmanual
>
>document necessity to force all build logs verbose
>
>--- devmanual/ebuild-writing/common-mistakes/text.xml
>+++ devmanual/ebuild-writing/common-mistakes/text.xml
>@@ -57,6 +57,37 @@
> </body>
> </section>
> 
>+<section>
>+<title>Build log not verbose</title>
>+<body>
>+When writing ebuilds you should always check the build log, because the build
>+system might ignore CC/CXX/LD/CFLAGS/LDFLAGS and such or add undesired flags
>+by default. In order to analyze this and have complete information in case
>+someone reports a bug for your package the <b>build log must always be verbose.</b>
>+<p>
>+There are several ways to fix non-verbose build logs depending on the build system:
>+</p>
>+<p>
>+For <c>cmake</c> based build systems it should be sufficient that the ebuild calls
>+cmake-utils_src_compile which picks up the cmake-utils.eclass variable 'CMAKE_VERBOSE=1'
>+by default. If you call emake directly for whatever reason you can do 'emake VERBOSE=1'
>+(note that cmake-utils_src_compile takes arguments as well which are passed to make).
>+</p>
>+
>+<p>
>+For <c>autotools</c> based build systems you can pass '--disable-silent-rules' to econf
>+until a new EAPI does that automatically. 'emake V=1' should also work.
>+</p>
>+
>+<p>
>+For custom Makefiles you often have to write a patch. Try to get upstream to include an
>+option like 'V=1' to enable full verbosity.
>+</p>
>+<note> If you notice non-verbose build log in any package open a bug and make it block the
>+tracker bug #429308</note>
>+</body>
>+</section>
>+
> </chapter>
> 
> </guide>

Actions: View | Diff

Attachments on bug 429744: 320234 | 320382