Try to make error output a little more understandable #385625 by Paul Varner.

Package-Manager: portage-2.2.0_alpha59/cvs/Linux x86_64

diff --git a/sys-apps/baselayout/ChangeLog b/sys-apps/baselayout/ChangeLog
index 0a33a294428cffc336b71666e6170dcadd85b4ba..950cfe54f1f45d341163cec0451d66cb3bb3dc87 100644 (file)
--- a/sys-apps/baselayout/ChangeLog
+++ b/sys-apps/baselayout/ChangeLog
@@ -1,10 +1,13 @@
 # ChangeLog for sys-apps/baselayout
 # Copyright 1999-2011 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/sys-apps/baselayout/ChangeLog,v 1.512 2011/10/03 03:59:17 vapier Exp $
+# $Header: /var/cvsroot/gentoo-x86/sys-apps/baselayout/ChangeLog,v 1.513 2011/10/04 18:36:31 vapier Exp $
 
 # See the rc-scripts ChangeLog in subversion for release info:
 # http://sources.gentoo.org/viewcvs.py/*checkout*/baselayout/trunk/ChangeLog
 
+  04 Oct 2011; Mike Frysinger <vapier@gentoo.org> baselayout-2.1.ebuild:
+  Try to make error output a little more understandable #385625 by Paul Varner.
+
 *baselayout-2.1 (03 Oct 2011)
 
   03 Oct 2011; Mike Frysinger <vapier@gentoo.org> +baselayout-2.1.ebuild:

diff --git a/sys-apps/baselayout/baselayout-2.1.ebuild b/sys-apps/baselayout/baselayout-2.1.ebuild
index eba860455d7cfdca3fbee89331e79916faedf49d..9b157fa7d29a51932129ce8c70b149233e4f47b9 100644 (file)
--- a/sys-apps/baselayout/baselayout-2.1.ebuild
+++ b/sys-apps/baselayout/baselayout-2.1.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2011 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/sys-apps/baselayout/baselayout-2.1.ebuild,v 1.1 2011/10/03 03:59:17 vapier Exp $
+# $Header: /var/cvsroot/gentoo-x86/sys-apps/baselayout/baselayout-2.1.ebuild,v 1.2 2011/10/04 18:36:31 vapier Exp $
 
 inherit eutils multilib
 
@@ -21,6 +21,17 @@ pkg_setup() {
 }
 
 # Create our multilib dirs - the Makefile has no knowledge of this
+multilib_warn() {
+       local syms=$1 dirs=$2 def_libdir=$3
+
+       [ -z "${syms}${dirs}" ] && return
+
+       ewarn "Your system profile has SYMLINK_LIB=${SYMLINK_LIB}, so that means"
+       if [ -z "${syms}" ] ; then
+               ewarn "you need to have these paths as symlinks to ${def_libdir}:"
+               ewarn "$1"
+       fi
+}
 multilib_layout() {
        local libdir libdirs=$(get_all_libdirs) def_libdir=$(get_abi_LIBDIR $DEFAULT_ABI)
        : ${libdirs:=lib}       # it isn't that we don't trust multilib.eclass...
@@ -38,16 +49,13 @@ multilib_layout() {
                        fi
                done
        done
-       if [ -n "${syms}" ] ; then
-               ewarn "Based on your profile, these paths should be directories:"
-               ewarn "${syms}"
-               ewarn "Failure to fix this may result in a broken system."
-               echo
-       fi
-       if [ -n "${dirs}" ] ; then
-               ewarn "Based on your profile, these paths should be symlinks:"
-               ewarn "${dirs}"
-               ewarn "Failure to fix this may result in a broken system."
+       if [ -n "${syms}${dirs}" ] ; then
+               ewarn "Your system profile has SYMLINK_LIB=${SYMLINK_LIB:-no}, so that means you need to"
+               ewarn "have these paths configured as follows:"
+               [ -n "${dirs}" ] && ewarn "symlinks to '${def_libdir}':${dirs}"
+               [ -n "${syms}" ] && ewarn "directories:${syms}"
+               ewarn "The ebuild will attempt to fix these, but only for trivial conversions."
+               ewarn "If things fail, you will need to manually create/move the directories."
                echo
        fi