diff options
author | Sam James <sam@gentoo.org> | 2021-03-21 02:36:43 +0000 |
---|---|---|
committer | Ulrich Müller <ulm@gentoo.org> | 2021-03-30 20:14:56 +0200 |
commit | d922c8c08a951c2a5f4c057cf6e1174d0a12f5a9 (patch) | |
tree | 152e6c8b7e7c5eabac1821e45a3ad8ccc4fe79c3 | |
parent | ebuild-writing/common-mistakes: mention "the compiler", not only GCC (diff) | |
download | devmanual-d922c8c0.tar.gz devmanual-d922c8c0.tar.bz2 devmanual-d922c8c0.zip |
ebuild-writing/common-mistakes: slight grammar fixes/phrasing changes
Signed-off-by: Sam James <sam@gentoo.org>
Signed-off-by: Ulrich Müller <ulm@gentoo.org>
-rw-r--r-- | ebuild-writing/common-mistakes/text.xml | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/ebuild-writing/common-mistakes/text.xml b/ebuild-writing/common-mistakes/text.xml index 9455688..05380bf 100644 --- a/ebuild-writing/common-mistakes/text.xml +++ b/ebuild-writing/common-mistakes/text.xml @@ -93,7 +93,7 @@ There are several ways to fix non-verbose build logs depending on the build syst '--disable-silent-rules' to econf, or use EAPI 5 where that argument is passed automatically. 'emake V=1' should also work.</li> - <li>For custom Makefiles you often have to write a patch. Try to get + <li>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.</li> <li>Note that when building Go manually outside of the eclass, you @@ -127,8 +127,8 @@ without purpose, e.g.: </p> <ul> <li> - new warnings on version bumps of GCC/glibc which the developer was not aware - of at the point of coding + new warnings on version bumps of GCC/glibc of which the developer was not + aware at the point of coding </li> <li> some autoconf checks will fail badly @@ -147,9 +147,9 @@ without purpose, e.g.: </li> </ul> <p> -Turning off "-Werror" we will still see the warnings, but there is no reason -that they cause compile failure. Also note that Portage already emits QA -notices about gcc warnings that can cause runtime breakage. +By turning off "-Werror", we will still see the warnings, but there is no reason +that they cause compile failure. Note that Portage already emits QA +notices about GCC warnings that can cause runtime breakage. </p> <p><b>How to fix</b></p> @@ -179,8 +179,8 @@ Always check that it's really gone in the build log. The compiler (e.g. GCC) can turn any specific warning into an error. A specific -Werror flag would be "-Werror=implicit-function-declaration" for example and will only affect warnings about implicit function declarations. It's -mostly safe to leave these untouched, cause they are pinned to this issue and -should not cause random build time breakage. Also, we can expect that upstream +mostly safe to leave these untouched, because they are pinned to this issue and +should not cause random build-time breakage. Also, we can expect that upstream did this on purpose to avoid known runtime errors and not just for testing their builds. However, you should check the specified warnings yourself or ask other developers if unsure. |