From 4a3ed90f2fe64bec43fcc8ba0a7df2897ebdb06b Mon Sep 17 00:00:00 2001
From: Eric Hameleers <alien@slackware.com>
Date: Tue, 27 Jan 2015 21:58:49 +0100
Subject: KDE 5 (Frameworks 5.6.0, Plasma 5.2.0, Applications 14.12.1) ready.

All packages have been built, this commit is a final SlackBuild polishing.
Expect a release for Slackware -current in my 'ktown' reository soon.
---
 README                                        | 12 ++++++++----
 kde/KDE.SlackBuild                            | 10 ++++++++++
 kde/modules/applications                      |  2 +-
 kde/slack-desc/polkit-kde-kcmodules-framework | 18 ++++++++++++++++++
 4 files changed, 37 insertions(+), 5 deletions(-)
 create mode 100644 kde/slack-desc/polkit-kde-kcmodules-framework

diff --git a/README b/README
index 2d7869c..d21e84c 100644
--- a/README
+++ b/README
@@ -1,8 +1,8 @@
 KDE 5_1501
 ==========
 
-This is going to be KDE 5 for Slackware-current, composed of Frameworks 5.6.0,
-Plasma 5.2.0 and Applications 14.12.1.
+This is KDE 5 for Slackware-current, composed of Frameworks 5.6.0, Plasma 5.2.0
+and Applications 14.12.1.
 
 Compared to Applications 14.12.0, the Applications 14.12.1 adds LTS releases of
 kdelibs, kdepimlibs, kdepim-runtime, kdepim and kde-workspace.
@@ -15,6 +15,8 @@ now found their rightful place): (lib)bluedevil, ksshaskpass, muon,
 sddm-kcm-module, kscreen2, polkit-framework, kde-gtk-config, kdecoration.
 And unfortunately, in order to be able to compile KWin, I had to add wayland
 to the "deps/" for this release of Plasma.
+I will not ship a 'muon' package since that is only meaningful on debian and
+derivatives like ubuntu.
 
 Note that there is still a lot of KDE 4 stuff in here, which is necessary to
 support all the Applications that have not yet been ported to KDE Frameworks 5.
@@ -58,6 +60,8 @@ Preliminary build procedure:
 - build the remaining KDE 5 stuff in the "kde/" directory, using "./KDE.SlackBui
 ld frameworks plasma plasma-extra applications"
 - check that everything built correctly
+- remove kde-workspace and re-install plasma-workspace (kde-workspace is only
+  used to build kdeartwork)
 - (optionally) build kdei
 - If you want to use SDDM as your graphical login manager, use the following
   patch to /etc/rc.d/rc.4:
@@ -115,8 +119,8 @@ DONE: khangman (needed libkeduvocdocument, was missing due to build order issue)
 DONE: parley (needed libkeduvocdocument, was missing due to build order issue)
 DONE: okular (needed libkscreen removed)
 DONE: kalgebra (needed a refreshed curses patch)
-SKIP: step (don't understand where the compilation error comes from)
+DONE: step (needed an update of eigne2 to compile successfully)
 
 ==============================================================================
-                     Eric Hameleers / alien at slackware dot com / 26-jan-2015
+                     Eric Hameleers / alien at slackware dot com / 27-jan-2015
 
diff --git a/kde/KDE.SlackBuild b/kde/KDE.SlackBuild
index 273fb90..80439ea 100755
--- a/kde/KDE.SlackBuild
+++ b/kde/KDE.SlackBuild
@@ -174,6 +174,16 @@ precheck() {
     echo "Multiply-defined package names: '$(echo $PKGDUP)'"
     RETVAL=1
   fi
+  unset PKGDUP
+
+  # Any source not being built modular (i.e. forgotten to add to modularize)?
+  for PSRC in $(cat modules/* | grep -v "^ *#" | grep -v "^$"); do
+    if ! grep -wq "^${PSRC}$" modularize ; then
+      echo "Not built modular: $PSRC"
+      RETVAL=1
+    fi
+  done
+  unset PSRC
 
   for MODULE in $(cat $CWD/modules/* | grep -v "^ *#") ; do
     # First find out if the pkg source is different from the actual pkg name:
diff --git a/kde/modules/applications b/kde/modules/applications
index 6cf4e6d..2345ab0 100644
--- a/kde/modules/applications
+++ b/kde/modules/applications
@@ -165,7 +165,7 @@ kwordquiz
 marble
 pairs
 rocs
-step
+#step
 # kdewebdev:
 kdewebdev
 
diff --git a/kde/slack-desc/polkit-kde-kcmodules-framework b/kde/slack-desc/polkit-kde-kcmodules-framework
new file mode 100644
index 0000000..91a3207
--- /dev/null
+++ b/kde/slack-desc/polkit-kde-kcmodules-framework
@@ -0,0 +1,18 @@
+# HOW TO EDIT THIS FILE:
+# The "handy ruler" below makes it easier to edit a package description.  Line
+# up the first '|' above the ':' following the base package name, and the '|'
+# on the right side marks the last column you can put a character in.  You must
+# make exactly 11 lines for the formatting to be correct.  It's also
+# customary to leave one space after the ':'.
+
+                             |-----handy-ruler-----------------------------------------------------|
+polkit-kde-kcmodules-framework: polkit-kde-kcmodules-framework (KDE polkit kcmodules)
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework: This package contains the kcm_polkit modules for KDE.
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework:
+polkit-kde-kcmodules-framework:
-- 
cgit v1.2.3