Package: ipppd
Version: 1:3.25+dfsg1-3+nmu2
Severity: serious
Tags: patch
User: debian...@lists.debian.org
Usertags: piuparts
Hi,
during a test with piuparts I noticed your package left unowned files on
the system after purge, which is a violation of policy ...
But the more serious bug is this link being created in the first place
due to a bug in the postinst script:
0m29.1s ERROR: FAIL: Package purging left files on system:
/isdnctrl -> isdnctrl0 not owned
I'm attaching a patch that fixes the link creation and cleans up the
misplaced one.
I intend to NMU isdnutils to unstable and get this fix into
wheezy-proposed-updates.
I'd prefer not to merge any other of the changes currently in
experimental into unstable at this time to give the fix that is intended
for stable a bit of testing in unstable first.
Andreas
ipppd_1:3.25+dfsg1-3+nmu2.log.gz
Description: GNU Zip compressed data
diff --git a/debian/changelog b/debian/changelog
index 3a2ef6d..4323b88 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,15 @@
+isdnutils (1:3.25+dfsg1-3.3) UNRELEASED; urgency=low
+
+ [ Andreas Beckmann ]
+ * Non-maintainer upload.
+ * ipppd.postinst: Create isdnctrl symlink as /dev/isdnctrl and cleanup
+possibly misplaced symlink /isdnctrl. (Closes: #xx)
+
+ [ Christoph Biedl ]
+ * unbreak debian/{ipppd,isdnlog}.config once more. Closes: #696660
+
+ -- Andreas Beckmann Sat, 01 Jun 2013 20:15:30 +0200
+
isdnutils (1:3.25+dfsg1-3+nmu2) unstable; urgency=medium
* Non-maintainer upload.
diff --git a/debian/gbp.conf b/debian/gbp.conf
new file mode 100644
index 000..0bb78a0
--- /dev/null
+++ b/debian/gbp.conf
@@ -0,0 +1,3 @@
+[DEFAULT]
+debian-branch = wheezy
+upstream-branch = dfsg
diff --git a/debian/ipppd.config b/debian/ipppd.config
index 33d9ae9..9d9fd50 100644
--- a/debian/ipppd.config
+++ b/debian/ipppd.config
@@ -37,6 +37,7 @@ if [ -s "/etc/isdn/device.$ipppd_if" -o -s "/etc/isdn/ipppd.$ipppd_if" ]; then
fi
db_input high ipppd/ispphone || true
+status=0
db_go || status=$?
if [ $status -eq 30 ]; then
db_stop
diff --git a/debian/ipppd.postinst b/debian/ipppd.postinst
index ccf442d..2702edb 100644
--- a/debian/ipppd.postinst
+++ b/debian/ipppd.postinst
@@ -27,8 +27,12 @@ case "$1" in
fi
# FIXME BUG! MAKEDEV should create the link by itself
if [ ! -d /dev/.udev/ ] && [ ! -d /run/udev ] ; then
- rm -f isdnctrl
- ln -s isdnctrl0 isdnctrl
+ rm -f /dev/isdnctrl
+ ln -s isdnctrl0 /dev/isdnctrl
+ fi
+ if [ -L /isdnctrl ]; then
+ # Cleanup possibly misplaced symlink created by 1:3.25+dfsg1-3+nmu2
+ rm /isdnctrl
fi
cd "$old_pwd"
forcerestart=false
diff --git a/debian/isdnlog.config b/debian/isdnlog.config
index 938a6da..68ef9a7 100644
--- a/debian/isdnlog.config
+++ b/debian/isdnlog.config
@@ -36,6 +36,7 @@ if [ -s /etc/isdn/isdn.conf ]; then
fi
fi
db_input high isdnlog/country || true
+status=0
db_go || status=$?
if [ $status -eq 30 ]; then
exit 1 # cancel selected
