tag 597940 patch
thanks

On 09/25/2010 09:52 AM, Patrick Matthäi wrote:
> I just wanted to remove this question again and I think this may be a
> good time to do it. :)

no; just fix your broken postinst. trivial patch attached.

-- 
Address:        Daniel Baumann, Burgunderstrasse 3, CH-4562 Biberist
Email:          daniel.baum...@panthera-systems.net
Internet:       http://people.panthera-systems.net/~daniel-baumann/
diff -Naurp apt-dater.orig/debian/apt-dater-host.postinst apt-dater/debian/apt-dater-host.postinst
--- apt-dater.orig/debian/apt-dater-host.postinst	2010-07-05 18:45:14.000000000 +0000
+++ apt-dater/debian/apt-dater-host.postinst	2010-09-25 08:00:51.861828186 +0000
@@ -8,11 +8,15 @@ set -e
 do_hostconfig() {
 	db_get apt-dater-host/assume_yes
 
-	ASSUME_YES=0
+	case "${RET}" in
+		true)
+			ASSUME_YES="1"
+			;;
 
-	if [ "$RET" = "true" ] ; then
-		ASSUME_YES=1
-	fi
+		false)
+			ASSUME_YES="0"
+			;;
+	esac
 
 	sed "s/^\$ASSUME_YES=.*/\$ASSUME_YES=${ASSUME_YES}/" -i "/etc/apt-dater-host.conf"
 }
diff -Naurp apt-dater.orig/debian/changelog apt-dater/debian/changelog
--- apt-dater.orig/debian/changelog	2010-07-05 18:45:13.000000000 +0000
+++ apt-dater/debian/changelog	2010-09-25 07:58:22.402831102 +0000
@@ -1,3 +1,10 @@
+apt-dater (0.8.4-2.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Using debconf correctly to respect user choices (Closes: #597940).
+
+ -- Daniel Baumann <pmatth...@debian.org>  Sat, 25 Sep 2010 07:57:51 +0000
+
 apt-dater (0.8.4-1) unstable; urgency=low
 
   * New upstream release.

Reply via email to