[debian-edu-commits] debian-edu/ 62/437: Document the policy issue with the kdm theme activation code.

[Mike Gabriel]

This is an automated email from the git hooks/post-receive script.

sunweaver pushed a commit to branch master
in repository debian-edu-artwork.

commit a31e1025d9e777cbb280f3cb50345d7958ff02dd
Author: Petter Reinholdtsen <pere at hungry.com>
Date:   Sun Sep 3 12:01:42 2006 +0000

    Document the policy issue with the kdm theme activation code.
---
 debian/debian-edu-artwork.postinst | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/debian/debian-edu-artwork.postinst b/debian/debian-edu-artwork.postinst
index 61b441c..fc04c29 100644
--- a/debian/debian-edu-artwork.postinst
+++ b/debian/debian-edu-artwork.postinst
@@ -9,8 +9,8 @@ change_ldm_background() {
         ldm-theme /usr/share/ldm/themes/debian-edu 80
 }
 
+# switch kdm theme to debian-edu
 change_kdm_theme() {
-    # switch kdm theme to debian-edu
     kdmrc=/etc/kde3/kdm/kdmrc
     if grep -q '^UseTheme=true' $kdmrc; then
 	if grep -q '^Theme=/usr/share/apps/kdm/themes/debian-edu' $kdmrc; then
@@ -20,6 +20,9 @@ change_kdm_theme() {
 	fi
     else
 	echo "Installing and enabling Debian Edu KDM theme ..."
+	# This is not policy compilant, as $kdmrc is a conffile in the
+	# kdm package, and we are editing it in a package maintainer
+	# script.  Need to come up with a better way to do it.
 	/usr/bin/update-ini-file $kdmrc 'X-*-Greeter' Theme /usr/share/apps/kdm/themes/debian-edu
 	/usr/bin/update-ini-file $kdmrc 'X-*-Greeter' UseTheme true
     fi

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-edu/debian-edu-artwork.git