[gentoo-commits] repo/gentoo:master commit in: eclass/

[Michael Palimaka]

commit:     ce4454e6c810066d91a65383aaaaff1402455154
Author:     Andreas Sturmlechner <andreas.sturmlechner <AT> gmail <DOT> com>
AuthorDate: Sun Nov 22 15:15:16 2015 +0000
Commit:     Michael Palimaka <kensington <AT> gentoo <DOT> org>
CommitDate: Tue Dec  8 13:59:21 2015 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ce4454e6

kde5.eclass: Extend KDE_TEST=forceoptional functionality, simplify

If KDE_TEST=forceoptional, it makes sense to disable test subdirs also
outside KDE categories. No change for misc ebuilds using kde5.eclass.

 eclass/kde5.eclass | 35 ++++++++++++++++-------------------
 1 file changed, 16 insertions(+), 19 deletions(-)

diff --git a/eclass/kde5.eclass b/eclass/kde5.eclass
index 0f50623..b2cc679 100644
--- a/eclass/kde5.eclass
+++ b/eclass/kde5.eclass
@@ -402,6 +402,10 @@ kde5_src_prepare() {
        # only enable handbook when required
        if ! use_if_iuse handbook ; then
                comment_add_subdirectory ${KDE_DOC_DIR}
+
+               if [[ ${KDE_HANDBOOK} = forceoptional ]] ; then
+                       punt_bogus_dep KF5 DocTools
+               fi
        fi
 
        # enable only the requested translations
@@ -435,21 +439,11 @@ kde5_src_prepare() {
                rm -rf po
        fi
 
-       # in frameworks, tests = manual tests so never
-       # build them
+       # in frameworks, tests = manual tests so never build them
        if [[ ${CATEGORY} = kde-frameworks ]]; then
                comment_add_subdirectory tests
        fi
 
-       if [[ ${CATEGORY} = kde-frameworks || ${CATEGORY} = kde-plasma || 
${CATEGORY} = kde-apps ]] ; then
-               # only build unit tests when required
-               if ! use_if_iuse test ; then
-                       comment_add_subdirectory autotests
-                       comment_add_subdirectory test
-                       comment_add_subdirectory tests
-               fi
-       fi
-
        case ${KDE_PUNT_BOGUS_DEPS} in
                false)  ;;
                *)
@@ -462,15 +456,18 @@ kde5_src_prepare() {
                        ;;
        esac
 
-       if [[ ${KDE_HANDBOOK} = forceoptional ]] ; then
-               if ! use_if_iuse handbook ; then
-                       punt_bogus_dep KF5 DocTools
-               fi
-       fi
-
-       if [[ ${KDE_TEST} = forceoptional ]] ; then
-               if ! use_if_iuse test ; then
+       # only build unit tests when required
+       if ! use_if_iuse test ; then
+               if [[ ${KDE_TEST} = forceoptional ]] ; then
                        punt_bogus_dep Qt5 Test
+                       # if forceoptional, also cover non-kde categories
+                       comment_add_subdirectory autotests
+                       comment_add_subdirectory test
+                       comment_add_subdirectory tests
+               elif [[ ${CATEGORY} = kde-frameworks || ${CATEGORY} = 
kde-plasma || ${CATEGORY} = kde-apps ]] ; then
+                       comment_add_subdirectory autotests
+                       comment_add_subdirectory test
+                       comment_add_subdirectory tests
                fi
        fi