diff options
author | Zac Medico <zmedico@gentoo.org> | 2008-07-15 10:45:42 +0000 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2008-07-15 10:45:42 +0000 |
commit | 6a30405e4d3bb39d00fd069e2521126520542d5c (patch) | |
tree | 8f86c1827f76675d7fedb5b72f54c92d3760f7dd | |
parent | 34261a71a119e7af9b972238afe7e6532c4a67e3 (diff) | |
download | portage-6a30405e4d3bb39d00fd069e2521126520542d5c.tar.gz portage-6a30405e4d3bb39d00fd069e2521126520542d5c.tar.bz2 portage-6a30405e4d3bb39d00fd069e2521126520542d5c.zip |
Since lots of packages will have to be fixed before stable portage can abort
with "this code is not 64bit clean", convert the die call to an eerror message
instructing the user to file a bug at bugs.gentoo.org. This should allow us
to stabilize portage-2.2 without having to ensure that the entire stable tree
is 64 bit clean first.
svn path=/main/trunk/; revision=11057
-rwxr-xr-x | bin/misc-functions.sh | 38 |
1 files changed, 29 insertions, 9 deletions
diff --git a/bin/misc-functions.sh b/bin/misc-functions.sh index 58507893c..b19a75b12 100755 --- a/bin/misc-functions.sh +++ b/bin/misc-functions.sh @@ -301,7 +301,7 @@ install_qa_check() { fi # Verify that the libtool files don't contain bogus $D entries. - abort="no" + local abort=no gentoo_bug=no for a in "${D}"usr/lib*/*.la ; do s=${a##*/} if grep -qs "${D}" "${a}" ; then @@ -339,18 +339,38 @@ install_qa_check() { done f=$(cat "${PORTAGE_LOG_FILE}" | check-implicit-pointer-usage.py) if [[ -n ${f} ]] ; then - vecho -ne '\a\n' - eqawarn "QA Notice: Package has poor programming practices which may compile" - eqawarn " but will almost certainly crash on 64bit architectures." - eqawarn "${f}" - vecho -ne '\a\n' + + # In the future this will be a forced "die". In preparation, + # increase the log level from "qa" to "eerror" so that people + # are aware this is a problem that must be fixed asap. + # just warn on 32bit hosts but bail on 64bit hosts case ${CHOST} in - alpha*|ia64*|powerpc64*|mips64*|sparc64*|x86_64*) die "this code is not 64bit clean";; - *) abort="yes";; + alpha*|ia64*|powerpc64*|mips64*|sparc64*|x86_64*) gentoo_bug=yes ;; esac + + abort=yes + + if [[ $gentoo_bug = yes ]] ; then + eerror + eerror "QA Notice: Package has poor programming practices which may compile" + eerror " but will almost certainly crash on 64bit architectures." + eerror + eerror "${f}" + eerror + eerror " Please file a bug about this at http://bugs.gentoo.org/" + eerror " with the maintaining herd of the package." + eerror + else + vecho -ne '\a\n' + eqawarn "QA Notice: Package has poor programming practices which may compile" + eqawarn " but will almost certainly crash on 64bit architectures." + eqawarn "${f}" + vecho -ne '\a\n' + fi + fi - if [[ ${abort} == "yes" ]] ; then + if [[ $abort = yes ]] && [[ $gentoo_bug != yes ]] ; then echo "Please do not file a Gentoo bug and instead" \ "report the above QA issues directly to the upstream" \ "developers of this software." | fmt -w 70 | \ |