Package: wpasupplicant
Version: 0.4.7-3
The following lintian report shows some quire serious problems:-
Now running lintian...
dpkg-source: warning: extracting unsigned source package (./dsc)
E: wpasupplicant: duplicate-updaterc.d-calls-in-postinst wpasupplicant
E: wpasupplicant: duplicate-updaterc.d-calls-in-postrm wpasupplicant
E: wpasupplicant: duplicate-conffile /etc/init.d/wpasupplicant
E: wpasupplicant: duplicate-conffile /etc/default/wpasupplicant
E: wpasupplicant: duplicate-conffile /etc/wpa_supplicant.conf
W: wpasupplicant: wrong-name-for-upstream-changelog
usr/share/doc/wpasupplicant/ChangeLog.gz
Finished running lintian.
Upon inspecting the source package I also noticed a few other things that could
be improved. Attached is a series of patches to address each issue that will be
briefly described below:-
00_debian-changelog.patch
The changelog entry briefly describing the following changes
01_use-upstream-manpages.patch
Remove debian manpages and used those that are (well) maintained upstream. No
need to duplicate efforts here imho.
02_wpagui-manpage-man8.patch
Install wpa_gui manpage to man8, as it is an administration tool. The other
manpages also live here.
03_cleanup-install-target.patch
Eliminate double handling of the binary installation. Don't use debian/tmp or
*.install files. Install binaries directly into their respective package
staging directory tree via existing debian/rules commands.
04_precise-qmake-cmd.patch
Explicitly define qmake-qt4 as the $(QMAKE) var. Fails to build on systems with
previous/parallel qt installations.
05_let-debhelper4-mark-conffiles.patch
The debhelper compat level used marks conffiles automatically.
06_let-dh_installinit-create-postrm.patch
The current postrm file contains the #DEBHELPER# token, which is automatically
replaced by the _exact_ same code in the rest of the script by dh_installinit.
This is not good.
07_dh_installinit-postinst-cleanup.patch
Set the update-rc.d options in debian/rules. Also eliminate duplication of code
automatically injected by dh_installinit.
08_install-upstream-changelog-correctly.patch
Use debian/rules to correctly install the upstream changelog.
09_add-watch-file.patch
Add a watch file to assist in tracking new upstream releases.
Thanks, Kel.
diff -Nru wpasupplicant-0.4.7.orig/debian/changelog wpasupplicant-0.4.7/debian/changelog
--- wpasupplicant-0.4.7.orig/debian/changelog 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/changelog 2006-02-19 14:34:27.083389648 +1000
@@ -1,3 +1,26 @@
+wpasupplicant (0.4.7-3.1) unstable; urgency=low
+
+ * Use upstream manpages.
+ * Install wpagui manpage to man8.
+ * Install packaged binaries directly into their package staging directories
+ instead of double handling them via debain/tmp. Remove wpagui.install and
+ wpasupplicant.install.
+ * Use qmake-qt4 directly, to avoid ftbfs on systems with other qt
+ versions installed.
+ * When using debhelper >= 4.1.0, conffiles are automatically marked, do not
+ duplicate this. Remove wpasupplicant.conffiles.
+ * Avoid duplication of update-rc.d commands in postrm by allowing
+ dh_installinit to automatically handle the situation. Remove
+ wpasupplicant.postrm.
+ * Use dh_installinit --update-rcd-params option when creating the
+ wpasupplicant.postinst script.
+ * Remove code duplicated by dh_installinit in wpasupplicant.postinst,
+ invoke-rc.d is added automatically.
+ * Install upstream changelog with correct name.
+ * Add watch file.
+
+ -- Kel Modderman <[EMAIL PROTECTED]> Sun, 19 Feb 2006 13:55:46 +1000
+
wpasupplicant (0.4.7-3) unstable; urgency=low
* Another brown paper bag release.
diff -Nru wpasupplicant-0.4.7.orig/debian/wpa_cli.1 wpasupplicant-0.4.7/debian/wpa_cli.1
--- wpasupplicant-0.4.7.orig/debian/wpa_cli.1 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpa_cli.1 1970-01-01 10:00:00.000000000 +1000
@@ -1,38 +0,0 @@
-.TH WPA_CLI 1 05.09.2004 "" "WPA client"
-.SH NAME
-wpa_cli \- WPA command line client
-.SH SYNOPSIS
-.B wpa_cli
-.I "[\-p<path>] [\-i<ifname>] [\-hvB] [\-a<actionfile>] [commands...]"
-.SH "DESCRIPTION"
-This manual page documents briefly the
-.BR wpa_cli
-utility.
-.SH OPTIONS
-A summary of commands follows. For a complete description, run
-.BR wpa_cli\ \-h
-from the command line. See the help output for a complete list of
-query commands.
-.TP
-.B \-p<path>
-Path to find control sockets.
-.TP
-.B \-B
-Run a daemon in the background.
-.TP
-.B \-a<actionfile>
-Run in daemon mode, executing the action file based on events from wpa_supplicant.
-.TP
-.B \-i<ifname>
-Interface to listen on. Default interface is the first found in the socket path.
-.TP
-.B \-h
-Help.
-.TP
-.B \-v
-Version information.
-.SH "SEE ALSO"
-wpa_supplicant(1), wpa_passphrase(1)
-.SH AUTHOR
-This manual page was written by Kyle McMartin <[EMAIL PROTECTED]> for
-the Debian GNU system (but may be used by others).
diff -Nru wpasupplicant-0.4.7.orig/debian/wpa_passphrase.1 wpasupplicant-0.4.7/debian/wpa_passphrase.1
--- wpasupplicant-0.4.7.orig/debian/wpa_passphrase.1 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpa_passphrase.1 1970-01-01 10:00:00.000000000 +1000
@@ -1,25 +0,0 @@
-.TH WPA_PASSPHRASE 1 05.09.2004 "" "WPA Passphrase"
-.SH NAME
-wpa_passphrase \- set WPA passphrase for a SSID
-.SH SYNOPSIS
-.B wpa_passphrase
-.I "<ssid> <passphrase>"
-.SH "DESCRIPTION"
-This manual page documents briefly the
-.BR wpa_passphrase
-utility.
-.SH OPTIONS
-A summary of commands follows. For a complete description, run
-.BR wpa_passphase
-from the command line.
-.TP
-.B <ssid>
-SSID to change passphrase on.
-.TP
-.B <passphrase>
-Passphrase to use on SSID. If passphrase is left out, it will be read from stdin.
-.SH "SEE ALSO"
-wpa_supplicant(1), wpa_cli(1)
-.SH AUTHOR
-This manual page was written by Kyle McMartin <[EMAIL PROTECTED]> for
-the Debian GNU system (but may be used by others).
diff -Nru wpasupplicant-0.4.7.orig/debian/wpa_supplicant.1 wpasupplicant-0.4.7/debian/wpa_supplicant.1
--- wpasupplicant-0.4.7.orig/debian/wpa_supplicant.1 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpa_supplicant.1 1970-01-01 10:00:00.000000000 +1000
@@ -1,63 +0,0 @@
-.TH WPA_SUPPLICANT 1 05.09.2004 "" "WPA Supplicant"
-.SH NAME
-wpa_supplicant \- WiFi Protected Access client daemon
-.SH SYNOPSIS
-.B wpa_supplicant
-.I "[\-BddehLqqvwW] \-i<ifname> \-c<conf> [\-D<driver>] [\-N \-i<ifname> \-c<conf> [\-D<driver>] ...]"
-.SH "DESCRIPTION"
-This manual page documents briefly the
-.BR wpa_supplicant
-daemon.
-.SH OPTIONS
-A summary of commands follows. For a complete description, run
-.BR wpa_supplicant
-from the command line.
-.TP
-.B \-i<iface>
-Interface to listen on.
-.TP
-.B \-c<conf>
-Path to configuration file, typically /etc/wpa_supplicant.conf. An
-example configuration file with many examples can be found in
-/usr/share/doc/wpasupplicant/examples/wpa_supplicant.conf.gz.
-.TP
-.B \-D<driver>
-Driver to use.
-.TP
-.B \-B
-Fork daemon to the background.
-.TP
-.B \-d
-Show debugging info (\-dd for even more).
-.TP
-.B \-K
-Include keys (passwords, etc.) in debug output.
-.TP
-.B \-t
-Include timestamp in debug messages.
-.TP
-.B \-h
-Show help text.
-.TP
-.B \-L
-Show license text.
-.TP
-.B \-q
-Decrease debugging info (\-qq for even less).
-.TP
-.B \-v
-Show version text.
-.TP
-.B \-w
-Wait for <iface> to be added, if needed.
-.TP
-.B \-W
-Wait for a control interface monitor before starting.
-.TP
-.B \-N
-Start describing a new interface.
-.SH "SEE ALSO"
-wpa_passphrase(1), wpa_cli(1)
-.SH AUTHOR
-This manual page was written by Kyle McMartin <[EMAIL PROTECTED]> for
-the Debian GNU system (but may be used by others).
diff -Nru wpasupplicant-0.4.7.orig/debian/wpasupplicant.manpages wpasupplicant-0.4.7/debian/wpasupplicant.manpages
--- wpasupplicant-0.4.7.orig/debian/wpasupplicant.manpages 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpasupplicant.manpages 2006-02-19 13:10:34.969387256 +1000
@@ -1,3 +1,3 @@
-debian/wpa_cli.1
-debian/wpa_supplicant.1
-debian/wpa_passphrase.1
+doc/docbook/wpa_cli.8
+doc/docbook/wpa_passphrase.8
+doc/docbook/wpa_supplicant.8
diff -Nru wpasupplicant-0.4.7.orig/debian/wpa_gui.1 wpasupplicant-0.4.7/debian/wpa_gui.1
--- wpasupplicant-0.4.7.orig/debian/wpa_gui.1 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpa_gui.1 1970-01-01 10:00:00.000000000 +1000
@@ -1,25 +0,0 @@
-.TH WPA_GUI 1 05.09.2004 "" "WPA GUI"
-.SH NAME
-wpa_gui \- WPA GUI client
-.SH SYNOPSIS
-.B wpa_gui
-.I "[\-p<ctrl iface dir>] [\-i<ctrl iface>]"
-.SH "DESCRIPTION"
-This manual page documents briefly the
-.BR wpa_gui
-utility.
-.SH OPTIONS
-A summary of parameters to wpa_gui follows.
-.TP
-.B \-p<ctrl iface dir>
-Path to where to find the control interface sockets. Typically this
-is /var/run/wpa_supplicant/
-.TP
-.B \-i<ctrl iface>
-Interface to listen on, ie, the socket you want to connect to, typically
-a file in /var/run/wpa_supplicant/
-.SH "SEE ALSO"
-wpa_supplicant(1), wpa_passphrase(1), wpa_cli(1)
-.SH AUTHOR
-This manual page was written by Kyle McMartin <[EMAIL PROTECTED]> for
-the Debian GNU system (but may be used by others).
diff -Nru wpasupplicant-0.4.7.orig/debian/wpa_gui.8 wpasupplicant-0.4.7/debian/wpa_gui.8
--- wpasupplicant-0.4.7.orig/debian/wpa_gui.8 1970-01-01 10:00:00.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpa_gui.8 2006-02-19 13:17:20.789693192 +1000
@@ -0,0 +1,25 @@
+.TH WPA_GUI 8 02.18.2006 "" "WPA GUI"
+.SH NAME
+wpa_gui \- WPA GUI client
+.SH SYNOPSIS
+.B wpa_gui
+.I "[\-p<ctrl iface dir>] [\-i<ctrl iface>]"
+.SH "DESCRIPTION"
+This manual page documents briefly the
+.BR wpa_gui
+utility.
+.SH OPTIONS
+A summary of parameters to wpa_gui follows.
+.TP
+.B \-p<ctrl iface dir>
+Path to where to find the control interface sockets. Typically this
+is /var/run/wpa_supplicant/
+.TP
+.B \-i<ctrl iface>
+Interface to listen on, ie, the socket you want to connect to, typically
+a file in /var/run/wpa_supplicant/
+.SH "SEE ALSO"
+wpa_supplicant(1), wpa_passphrase(1), wpa_cli(1)
+.SH AUTHOR
+This manual page was written by Kyle McMartin <[EMAIL PROTECTED]> for
+the Debian GNU system (but may be used by others).
diff -Nru wpasupplicant-0.4.7.orig/debian/wpagui.manpages wpasupplicant-0.4.7/debian/wpagui.manpages
--- wpasupplicant-0.4.7.orig/debian/wpagui.manpages 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpagui.manpages 2006-02-19 13:17:30.027288864 +1000
@@ -1 +1 @@
-debian/wpa_gui.1
+debian/wpa_gui.8
diff -Nru wpasupplicant-0.4.7.orig/debian/rules wpasupplicant-0.4.7/debian/rules
--- wpasupplicant-0.4.7.orig/debian/rules 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/rules 2006-02-19 13:36:07.607390864 +1000
@@ -47,15 +47,12 @@
dh_clean -k
dh_installdirs
- mkdir -p debian/tmp/etc
- mkdir -p debian/tmp/usr/sbin
-
install -p wpa_cli wpa_passphrase wpa_supplicant \
- debian/tmp/usr/sbin
+ debian/wpasupplicant/usr/sbin
install --mode=644 -p debian/wpa_supplicant.defconf \
- debian/tmp/etc/wpa_supplicant.conf
+ debian/wpasupplicant/etc/wpa_supplicant.conf
- install -p wpa_gui-qt4/wpa_gui debian/tmp/usr/sbin
+ install -p wpa_gui-qt4/wpa_gui debian/wpagui/usr/sbin
dh_install
diff -Nru wpasupplicant-0.4.7.orig/debian/wpagui.install wpasupplicant-0.4.7/debian/wpagui.install
--- wpasupplicant-0.4.7.orig/debian/wpagui.install 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpagui.install 1970-01-01 10:00:00.000000000 +1000
@@ -1 +0,0 @@
-debian/tmp/usr/sbin/wpa_gui /usr/sbin
diff -Nru wpasupplicant-0.4.7.orig/debian/wpasupplicant.install wpasupplicant-0.4.7/debian/wpasupplicant.install
--- wpasupplicant-0.4.7.orig/debian/wpasupplicant.install 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpasupplicant.install 1970-01-01 10:00:00.000000000 +1000
@@ -1,4 +0,0 @@
-debian/tmp/etc/wpa_supplicant.conf /etc
-debian/tmp/usr/sbin/wpa_supplicant /usr/sbin
-debian/tmp/usr/sbin/wpa_cli /usr/sbin
-debian/tmp/usr/sbin/wpa_passphrase /usr/sbin
diff -Nru wpasupplicant-0.4.7.orig/debian/rules wpasupplicant-0.4.7/debian/rules
--- wpasupplicant-0.4.7.orig/debian/rules 2006-02-19 13:18:37.861976000 +1000
+++ wpasupplicant-0.4.7/debian/rules 2006-02-19 13:22:39.943174488 +1000
@@ -20,7 +20,7 @@
WPASUP=wpasupplicant
WPAGUI=wpagui
-QMAKE=qmake
+QMAKE=qmake-qt4
# This has to be exported to make some magic below work.
export DH_OPTIONS
diff -Nru wpasupplicant-0.4.7.orig/debian/wpasupplicant.conffiles wpasupplicant-0.4.7/debian/wpasupplicant.conffiles
--- wpasupplicant-0.4.7.orig/debian/wpasupplicant.conffiles 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpasupplicant.conffiles 1970-01-01 10:00:00.000000000 +1000
@@ -1,3 +0,0 @@
-/etc/default/wpasupplicant
-/etc/wpa_supplicant.conf
-/etc/init.d/wpasupplicant
diff -Nru wpasupplicant-0.4.7.orig/debian/wpasupplicant.postrm wpasupplicant-0.4.7/debian/wpasupplicant.postrm
--- wpasupplicant-0.4.7.orig/debian/wpasupplicant.postrm 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpasupplicant.postrm 1970-01-01 10:00:00.000000000 +1000
@@ -1,10 +0,0 @@
-#!/bin/sh
-# postrm for wpasupplicant
-
-if [ "$1" = "purge" ] ; then
- update-rc.d wpasupplicant remove >/dev/null
-fi
-
-#DEBHELPER#
-
-exit 0
diff -Nru wpasupplicant-0.4.7.orig/debian/rules wpasupplicant-0.4.7/debian/rules
--- wpasupplicant-0.4.7.orig/debian/rules 2006-02-19 13:37:04.439751000 +1000
+++ wpasupplicant-0.4.7/debian/rules 2006-02-19 13:46:32.768351952 +1000
@@ -65,7 +65,7 @@
dh_installchangelogs
dh_installman
dh_installdocs
- dh_installinit
+ dh_installinit --update-rcd-params="start 38 S . stop 34 0 6 ."
dh_strip
dh_link
dh_compress
diff -Nru wpasupplicant-0.4.7.orig/debian/wpasupplicant.postinst wpasupplicant-0.4.7/debian/wpasupplicant.postinst
--- wpasupplicant-0.4.7.orig/debian/wpasupplicant.postinst 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpasupplicant.postinst 2006-02-19 13:47:58.459324944 +1000
@@ -8,14 +8,6 @@
set -e
-case "$1" in
- abort*upgrade) exit 0;;
- abort*remove) exit 0;;
- abort*deconfigure) exit 0;;
- configure) ;;
- *) exit 0;
-esac
-
if [ -x "/etc/init.d/wpasupplicant" ]; then
# remove legacy run levels
if [ -x "/etc/rcS.d/S21wpasupplicant" ]; then
@@ -25,14 +17,6 @@
if [ -x "/etc/rc2.d/S20wpasupplicant" ]; then
update-rc.d -f wpasupplicant remove > /dev/null
fi
-
- update-rc.d wpasupplicant start 38 S . stop 34 0 6 . >/dev/null
-fi
-
-if [ -x /usr/sbin/invoke-rc.d ]; then
- invoke-rc.d wpasupplicant start 2>&1 3>&1 || true
-else
- /etc/init.d/wpasupplicant start 2>&1 3>&1 || true
fi
#DEBHELPER#
diff -Nru wpasupplicant-0.4.7.orig/debian/rules wpasupplicant-0.4.7/debian/rules
--- wpasupplicant-0.4.7.orig/debian/rules 2006-02-19 13:46:32.768351000 +1000
+++ wpasupplicant-0.4.7/debian/rules 2006-02-19 13:52:17.104004976 +1000
@@ -62,7 +62,7 @@
dh_testdir
dh_testroot
dh_installexamples
- dh_installchangelogs
+ dh_installchangelogs ChangeLog
dh_installman
dh_installdocs
dh_installinit --update-rcd-params="start 38 S . stop 34 0 6 ."
diff -Nru wpasupplicant-0.4.7.orig/debian/wpasupplicant.docs wpasupplicant-0.4.7/debian/wpasupplicant.docs
--- wpasupplicant-0.4.7.orig/debian/wpasupplicant.docs 2006-02-19 10:55:12.000000000 +1000
+++ wpasupplicant-0.4.7/debian/wpasupplicant.docs 1970-01-01 10:00:00.000000000 +1000
@@ -1 +0,0 @@
-ChangeLog
diff -Nru wpasupplicant-0.4.7.orig/debian/watch wpasupplicant-0.4.7/debian/watch
--- wpasupplicant-0.4.7.orig/debian/watch 1970-01-01 10:00:00.000000000 +1000
+++ wpasupplicant-0.4.7/debian/watch 2006-02-19 14:42:27.620336928 +1000
@@ -0,0 +1,2 @@
+version=3
+http://hostap.epitest.fi/releases/wpa_supplicant-(.*)\.tar\.gz debian