summaryrefslogtreecommitdiffstats
path: root/bin
diff options
context:
space:
mode:
authorZac Medico <zmedico@gentoo.org>2008-07-15 10:45:42 +0000
committerZac Medico <zmedico@gentoo.org>2008-07-15 10:45:42 +0000
commit6a30405e4d3bb39d00fd069e2521126520542d5c (patch)
tree8f86c1827f76675d7fedb5b72f54c92d3760f7dd /bin
parent34261a71a119e7af9b972238afe7e6532c4a67e3 (diff)
downloadportage-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
Diffstat (limited to 'bin')
-rwxr-xr-xbin/misc-functions.sh38
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 | \