commit:     c5f5858e6ed8800f1b44d3a24296a66b3e79527e
Author:     Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
AuthorDate: Sun Aug 23 16:25:39 2020 +0000
Commit:     Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
CommitDate: Sun Aug 23 19:05:30 2020 +0000
URL:        https://gitweb.gentoo.org/proj/kde-sunset.git/commit/?id=c5f5858e

kde.eclass, kde-functions.eclass: Properly fix KDEBASE to address the eloquent 
remark in 7ba60b69

Signed-off-by: Andreas Sturmlechner <asturm <AT> gentoo.org>

 eclass/kde-functions.eclass | 17 ++++++++++-------
 eclass/kde.eclass           |  4 ++--
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/eclass/kde-functions.eclass b/eclass/kde-functions.eclass
index b9137fa2..9c9506db 100644
--- a/eclass/kde-functions.eclass
+++ b/eclass/kde-functions.eclass
@@ -387,7 +387,7 @@ need-kde() {
        else
                x_DEPEND="${DEPEND}"
        fi
-       if [[ -n "${KDEBASE}" ]]; then
+       if [[ "${KDEBASE}" == "true" ]]; then
                # If we're a kde-base package, we need at least our own version 
of kdelibs.
                # Note: we only set RDEPEND if it is already set, otherwise
                # we break packages relying on portage copying RDEPEND from 
DEPEND.
@@ -404,7 +404,7 @@ need-kde() {
        qtver-from-kdever ${KDEVER}
        need-qt ${selected_version}
 
-       if [[ -n "${KDEBASE}" ]]; then
+       if [[ "${KDEBASE}" == "true" ]]; then
                SLOT="$KDEMAJORVER.$KDEMINORVER"
        else
                : ${SLOT="0"}
@@ -457,7 +457,7 @@ set-kdedir() {
        if [[ -n "$KDEPREFIX" ]]; then
                export PREFIX="$KDEPREFIX"
        else
-               if  [[ -z "$KDEBASE" ]]; then
+               if  [[ -z ${KDEBASE} || ${KDEBASE} != "true" ]]; then
                        PREFIX="/usr/kde/3.5"
                else
                        case $KDEMAJORVER.$KDEMINORVER in
@@ -472,7 +472,7 @@ set-kdedir() {
        if [[ -n "$KDELIBSDIR" ]]; then
                export KDEDIR="$KDELIBSDIR"
        else
-               if [[ -z "$KDEBASE" ]]; then
+               if  [[ -z ${KDEBASE} || ${KDEBASE} != "true" ]]; then
                        # find the latest kdelibs installed
                        for x in /usr/kde/{svn,3.5} "${PREFIX}" \
                                "${KDE3LIBSDIR}" "${KDELIBSDIR}" "${KDE3DIR}" 
"${KDEDIR}" /usr/kde/*; do
@@ -662,9 +662,12 @@ postprocess_desktop_entries() {
        validate_desktop_entries "${PREFIX}"/share/applications
 }
 
-# is this a kde-base ebuid? CYKER - KLUDGE KLUDGE KLUDGE FUCK YOU kde-apps
+# is this ebuild part of the KDE SC? kde-base/ or kde-apps/ are only hints
 if [[ "${CATEGORY}" == "kde-base" || "${CATEGORY}" == "kde-apps" ]]; then
        debug-print "${ECLASS}: KDEBASE ebuild recognized"
-       export KDEBASE="true"
-       export KDEREVISION
+       KDEBASE=${KDEBASE:=true}
+       if [[ ${KDEBASE} == "true" ]]; then
+               export KDEBASE
+               export KDEREVISION
+       fi
 fi

diff --git a/eclass/kde.eclass b/eclass/kde.eclass
index c0c636c4..8bf71e2a 100644
--- a/eclass/kde.eclass
+++ b/eclass/kde.eclass
@@ -173,7 +173,7 @@ kde_src_prepare() {
                        for f in "${PATCHDIR}"/${p}-${PV}-*{diff,patch}; do
                                [[ -e ${f} ]] && PATCHES+=("${f}")
                        done
-                       if [[ -n "${KDEBASE}" ]]; then
+                       if [[ "${KDEBASE}" == "true" ]]; then
                                for f in 
"${PATCHDIR}"/${p}-${SLOT}-*{diff,patch}; do
                                        [[ -e ${f} ]] && PATCHES+=("${f}")
                                done
@@ -563,7 +563,7 @@ kde_src_install() {
        shift
        done
 
-       if [[ -n ${KDEBASE} && "${PN}" != "arts" && -d 
"${D}"/usr/share/doc/${PF} ]]; then
+       if [[ ${KDEBASE} == "true" && "${PN}" != "arts" && -d 
"${D}"/usr/share/doc/${PF} ]]; then
                # work around bug #97196
                dodir /usr/share/doc/kde && \
                        mv "${D}"/usr/share/doc/${PF} "${D}"/usr/share/doc/kde/ 
|| \

Reply via email to