[gentoo-commits] gentoo-x86 commit in eclass: kde4-base.eclass kde4-functions.eclass

[Tomas Chvatal (scarabeus)]

scarabeus    09/02/10 20:07:24

  Modified:             kde4-base.eclass kde4-functions.eclass
  Log:
  Make NEED_KDE=none packages pick correct kdedir :] Fixed thanks to Tomáš Caithaml and pva for helping with this. Per bug #257616.

Revision  Changes    Path
1.29                 eclass/kde4-base.eclass

file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/eclass/kde4-base.eclass?rev=1.29&view=markup
plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/eclass/kde4-base.eclass?rev=1.29&content-type=text/plain
diff : http://sources.gentoo.org/viewcvs.py/gentoo-x86/eclass/kde4-base.eclass?r1=1.28&r2=1.29

Index: kde4-base.eclass
===================================================================
RCS file: /var/cvsroot/gentoo-x86/eclass/kde4-base.eclass,v
retrieving revision 1.28
retrieving revision 1.29
diff -u -r1.28 -r1.29
--- kde4-base.eclass	2 Feb 2009 21:35:08 -0000	1.28
+++ kde4-base.eclass	10 Feb 2009 20:07:24 -0000	1.29
@@ -1,6 +1,6 @@
 # Copyright 2007-2008 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/eclass/kde4-base.eclass,v 1.28 2009/02/02 21:35:08 scarabeus Exp $
+# $Header: /var/cvsroot/gentoo-x86/eclass/kde4-base.eclass,v 1.29 2009/02/10 20:07:24 scarabeus Exp $
 
 # @ECLASS: kde4-base.eclass
 # @MAINTAINER:
@@ -439,17 +439,15 @@
 
 	# Search for best suitable kde installation for misc kde package.
 	# Computation based on NEED_KDE and KDE_MINIMAL
-	get_latest_kdedir
+	[[ ${NEED_KDE} = latest || ${NEED_KDE} = none ]] && get_latest_kdedir
 
-	if [[ $NEED_KDE != none ]]; then
-		# Set PREFIX
-		if use kdeprefix; then
-			KDEDIR="/usr/kde/${_kdedir}"
-			KDEDIRS="/usr/local/:/usr:${KDEDIR}"
-		else
-			KDEDIR="/usr"
-			KDEDIRS="/usr/local/:/usr"
-		fi
+	# Set PREFIX
+	if use kdeprefix; then
+		KDEDIR="/usr/kde/${_kdedir}"
+		KDEDIRS="/usr/local/:/usr:${KDEDIR}"
+	else
+		KDEDIR="/usr"
+		KDEDIRS="/usr/local/:/usr"
 	fi
 	# Set the prefix based on KDEDIR
 	# Make it a consequence of kdeprefix



1.12                 eclass/kde4-functions.eclass

file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/eclass/kde4-functions.eclass?rev=1.12&view=markup
plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/eclass/kde4-functions.eclass?rev=1.12&content-type=text/plain
diff : http://sources.gentoo.org/viewcvs.py/gentoo-x86/eclass/kde4-functions.eclass?r1=1.11&r2=1.12

Index: kde4-functions.eclass
===================================================================
RCS file: /var/cvsroot/gentoo-x86/eclass/kde4-functions.eclass,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -r1.11 -r1.12
--- kde4-functions.eclass	12 Jan 2009 19:40:34 -0000	1.11
+++ kde4-functions.eclass	10 Feb 2009 20:07:24 -0000	1.12
@@ -1,6 +1,6 @@
 # Copyright 1999-2008 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/eclass/kde4-functions.eclass,v 1.11 2009/01/12 19:40:34 scarabeus Exp $
+# $Header: /var/cvsroot/gentoo-x86/eclass/kde4-functions.eclass,v 1.12 2009/02/10 20:07:24 scarabeus Exp $
 
 # @ECLASS: kde4-functions.eclass
 # @MAINTAINER:
@@ -226,45 +226,43 @@
 # We can check for kdelibs because it is the most basic package; no KDE package
 # working without it. This might be changed in future.
 get_latest_kdedir() {
-	if [[ $NEED_KDE = latest && $KDEBASE != kde-base ]]; then
-		case ${KDE_WANTED} in
-			# note this will need to be updated as stable moves and so on
-			live)
-				_versions="9999 4.1.69 4.1.0"
-				;;
-			snapshot)
-				_versions="4.1.69 4.1.0 9999"
+	case ${KDE_WANTED} in
+		# note this will need to be updated as stable moves and so on
+		live)
+			_versions="9999 4.1.69 4.1.0"
+			;;
+		snapshot)
+			_versions="4.1.69 4.1.0 9999"
+			;;
+		testing)
+			_versions="4.1.0 4.1.69 9999"
+			;;
+		stable)
+			_versions="4.1.0 4.1.69 9999"
+			;;
+		*) die "KDE_WANTED=${KDE_WANTED} not supported here." ;;
+	esac
+	# check if exists and fallback as we go
+	for X in ${_versions}; do
+		if has_version ">=kde-base/kdelibs-${X}"; then
+			# figure out which X we are in and set it into _kdedir
+			case ${X} in
+				# also keep track here same for kde_wanted
+				9999)
+					_kdedir="live"
+					break
 				;;
-			testing)
-				_versions="4.1.0 4.1.69 9999"
+				4.1.69)
+					_kdedir="4.2"
+					break
 				;;
-			stable)
-				_versions="4.1.0 4.1.69 9999"
+				4.1.0)
+					_kdedir="4.1"
+					break
 				;;
-			*) die "KDE_WANTED=${KDE_WANTED} not supported here." ;;
-		esac
-		# check if exists and fallback as we go
-		for X in ${_versions}; do
-			if has_version ">=kde-base/kdelibs-${X}"; then
-				# figure out which X we are in and set it into _kdedir
-				case ${X} in
-					# also keep track here same for kde_wanted
-					9999)
-						_kdedir="live"
-						break
-					;;
-					4.1.69)
-						_kdedir="4.2"
-						break
-					;;
-					4.1.0)
-						_kdedir="4.1"
-						break
-					;;
-				esac
-			fi
-		done
-	fi
+			esac
+		fi
+	done
 }
 
 # @FUNCTION: migrate_store_dir