Web lists-archives.com

intended MBF: wrong redirections in maintainer scripts




Hi,

as announced in our talk at debconf'18 [1] we intend a MBF about wrong
redirections in maintainer scripts. In general these are of the form

  foo 2>&1 1> /dev/null

Here it was probably intended to send both stderr and stdout to /dev/null.
In reality the effect of this is to send only 1 to /dev/null, and to send
2 to the file that 1 was sent to before that line. We found 128 occurrences
of this problem in sid in 110 different packages. A list of packages
and of maintainers is attached.

Do you think that a MBF is appropriate for this? If yes then it deserves
probably only severity=minor. We will of course also submit a wishlist
bug for this against lintian.

-Nicolas and Ralf.

[1] https://debconf18.debconf.org/talks/90-mining-debian-maintainer-scripts/
aewm++-goodies	1.0-10	postrm
bacula-common	9.0.8-1	preinst
cyrus-common	2.5.10-3+b1	postinst
dict-freedict-afr-deu	2016.12.12-1	postinst
dict-freedict-afr-eng	2016.12.12-1	postinst
dict-freedict-all	2016.12.12-1	postinst
dict-freedict-ara-eng	2016.12.12-1	postinst
dict-freedict-bre-fra	2016.12.12-1	postinst
dict-freedict-ces-eng	2016.12.12-1	postinst
dict-freedict-ckb-kmr	2016.12.12-1	postinst
dict-freedict-cym-eng	2016.12.12-1	postinst
dict-freedict-dan-eng	2016.12.12-1	postinst
dict-freedict-deu-eng	2016.12.12-1	postinst
dict-freedict-deu-fra	2016.12.12-1	postinst
dict-freedict-deu-ita	2016.12.12-1	postinst
dict-freedict-deu-kur	2016.12.12-1	postinst
dict-freedict-deu-nld	2016.12.12-1	postinst
dict-freedict-deu-por	2016.12.12-1	postinst
dict-freedict-deu-swe	2016.12.12-1	postinst
dict-freedict-deu-tur	2016.12.12-1	postinst
dict-freedict-eng-afr	2016.12.12-1	postinst
dict-freedict-eng-ara	2016.12.12-1	postinst
dict-freedict-eng-ces	2016.12.12-1	postinst
dict-freedict-eng-cym	2016.12.12-1	postinst
dict-freedict-eng-deu	2016.12.12-1	postinst
dict-freedict-eng-ell	2016.12.12-1	postinst
dict-freedict-eng-fra	2016.12.12-1	postinst
dict-freedict-eng-gle	2016.12.12-1	postinst
dict-freedict-eng-hin	2016.12.12-1	postinst
dict-freedict-eng-hrv	2016.12.12-1	postinst
dict-freedict-eng-hun	2016.12.12-1	postinst
dict-freedict-eng-ita	2016.12.12-1	postinst
dict-freedict-eng-lat	2016.12.12-1	postinst
dict-freedict-eng-lit	2016.12.12-1	postinst
dict-freedict-eng-nld	2016.12.12-1	postinst
dict-freedict-eng-pol	2016.12.12-1	postinst
dict-freedict-eng-por	2016.12.12-1	postinst
dict-freedict-eng-rom	2016.12.12-1	postinst
dict-freedict-eng-rus	2016.12.12-1	postinst
dict-freedict-eng-spa	2016.12.12-1	postinst
dict-freedict-eng-srp	2016.12.12-1	postinst
dict-freedict-eng-swe	2016.12.12-1	postinst
dict-freedict-eng-swh	2016.12.12-1	postinst
dict-freedict-eng-tur	2016.12.12-1	postinst
dict-freedict-fra-bre	2016.12.12-1	postinst
dict-freedict-fra-deu	2016.12.12-1	postinst
dict-freedict-fra-eng	2016.12.12-1	postinst
dict-freedict-fra-nld	2016.12.12-1	postinst
dict-freedict-gla-deu	2016.12.12-1	postinst
dict-freedict-gle-eng	2016.12.12-1	postinst
dict-freedict-gle-pol	2016.12.12-1	postinst
dict-freedict-hrv-eng	2016.12.12-1	postinst
dict-freedict-hun-eng	2016.12.12-1	postinst
dict-freedict-isl-eng	2016.12.12-1	postinst
dict-freedict-ita-deu	2016.12.12-1	postinst
dict-freedict-ita-eng	2016.12.12-1	postinst
dict-freedict-jpn-deu	2016.12.12-1	postinst
dict-freedict-jpn-eng	2016.12.12-1	postinst
dict-freedict-jpn-fra	2016.12.12-1	postinst
dict-freedict-jpn-rus	2016.12.12-1	postinst
dict-freedict-kha-deu	2016.12.12-1	postinst
dict-freedict-kha-eng	2016.12.12-1	postinst
dict-freedict-kur-deu	2016.12.12-1	postinst
dict-freedict-kur-eng	2016.12.12-1	postinst
dict-freedict-kur-tur	2016.12.12-1	postinst
dict-freedict-lat-deu	2016.12.12-1	postinst
dict-freedict-lat-eng	2016.12.12-1	postinst
dict-freedict-lit-eng	2016.12.12-1	postinst
dict-freedict-mkd-bul	2016.12.12-1	postinst
dict-freedict-nld-deu	2016.12.12-1	postinst
dict-freedict-nld-eng	2016.12.12-1	postinst
dict-freedict-nld-fra	2016.12.12-1	postinst
dict-freedict-nno-nob	2016.12.12-1	postinst
dict-freedict-oci-cat	2016.12.12-1	postinst
dict-freedict-pol-gle	2016.12.12-1	postinst
dict-freedict-por-deu	2016.12.12-1	postinst
dict-freedict-por-eng	2016.12.12-1	postinst
dict-freedict-san-deu	2016.12.12-1	postinst
dict-freedict-slk-eng	2016.12.12-1	postinst
dict-freedict-spa-ast	2016.12.12-1	postinst
dict-freedict-spa-eng	2016.12.12-1	postinst
dict-freedict-spa-por	2016.12.12-1	postinst
dict-freedict-srp-eng	2016.12.12-1	postinst
dict-freedict-swe-deu	2016.12.12-1	postinst
dict-freedict-swe-eng	2016.12.12-1	postinst
dict-freedict-swh-eng	2016.12.12-1	postinst
dict-freedict-swh-pol	2016.12.12-1	postinst
dict-freedict-tur-deu	2016.12.12-1	postinst
dict-freedict-tur-eng	2016.12.12-1	postinst
gwhois	20120626-1.2	postinst
gwhois	20120626-1.2	prerm
libmkl-dev	2018.3.222-1	postinst
libmkl-rt	2018.3.222-1	postinst
lilo	1:24.2-3	postinst
linuxdoc-tools	0.9.72-7	postinst
linuxdoc-tools	0.9.72-7	postrm
mimedefang	2.84-2	postinst
miredo	1.2.6-4	postrm
miredo-server	1.2.6-4	postrm
muroard	0.1.14-5	postrm
mysql-server-5.7	5.7.22-1	postinst
needrestart-session	0.3-5	postrm
prads	0.3.3-1+b1	postinst
prelude-correlator	4.1.1-2	postinst
prelude-manager	4.1.1-2	postinst
pyca	20031119-0	postinst
pyca	20031119-0	postrm
rdnssd	1.0.3-3	postrm
roaraudio	1.0~beta11-10	postrm
roarplaylistd	0.1.9-6	postrm
sagan	1.2.0-1	postinst
tenshi	0.13-2	postinst
tenshi	0.13-2	postrm
varnish	5.2.1-1+b1	postinst
Agustin Martin Domingo <agmartin@xxxxxxxxxx>
   linuxdoc-tools (U)

Anthony Prades <toony.debian@xxxxxxxxxxxx>
   cyrus-imapd (U)

Bernhard Schmidt <berni@xxxxxxxxxx>
   ndisc6

Carsten Leonhardt <leo@xxxxxxxxxx>
   bacula (U)

Chris Boyle <cmb@xxxxxxxxxx>
   aewm++-goodies

Christoph Martin <martin@xxxxxxxxxxxx>
   mimedefang

Clint Byrum <clint@xxxxxxxxxx>
   mysql-5.7 (U)

Debian Bacula Team <pkg-bacula-devel@xxxxxxxxxxxxxxxxxxxxxxx>
   bacula

Debian Cyrus Team <pkg-cyrus-imapd-debian-devel@xxxxxxxxxxxxxxxxxxxxxxx>
   cyrus-imapd

Debian MySQL Maintainers <pkg-mysql-maint@xxxxxxxxxxxxxxxxxxxxxxx>
   mysql-5.7

Debian Science Maintainers <debian-science-maintainers@xxxxxxxxxxxxxxxxxxxxxxx>
   intel-mkl

Debian XML/SGML Group <debian-xml-sgml-pkgs@xxxxxxxxxxxxxxxxxxxxxxx>
   linuxdoc-tools

Emanuele Rocca <ema@xxxxxxxxxx>
   varnish (U)

Henrique de Moraes Holschuh <hmh@xxxxxxxxxx>
   cyrus-imapd (U)

Ignace Mouzannar <mouzannar@xxxxxxxxx>
   tenshi

James Page <jamespage@xxxxxxxxxx>
   mysql-5.7 (U)

Jan Wagner <waja@xxxxxxxxxxx>
   varnish (U)

Joachim Wiedorn <joodebian@xxxxxxxxx>
   lilo

Juliane Holzt <debian@xxxxxxxxxxx>
   gwhois

Kacper Wysocki <comotion@xxxxxxxxx>
   prads (U)

Lars Bahner <bahner@xxxxxxxxxx>
   pyca
   varnish (U)

Lars Tangvald <lars.tangvald@xxxxxxxxxx>
   mysql-5.7 (U)

Mo Zhou <cdluminate@xxxxxxxxx>
   intel-mkl (U)

Norbert Tretkowski <norbert@xxxxxxxxxxxxx>
   mysql-5.7 (U)

Ondřej Surý <ondrej@xxxxxxxxxx>
   cyrus-imapd (U)

Patrick Matthäi <pmatthaei@xxxxxxxxxx>
   muroard
   needrestart-session
   roaraudio
   roarplaylistd

Pierre Chifflier <pollux@xxxxxxxxxx>
   prelude-correlator
   prelude-manager
   sagan

Prads package developers <prads-devel@xxxxxxxxxxxxxxxxxx>
   prads

Robie Basak <robie@xxxxxxxxxxxxxxxx>
   mysql-5.7 (U)

Sebastian Humenda <shumenda@xxxxxx>
   freedict

Stig Sandbeck Mathisen <ssm@xxxxxxxxxx>
   prads (U)
   varnish (U)

Thomas Andrejak <thomas.andrejak@xxxxxxxxx>
   prelude-correlator (U)
   prelude-manager (U)

Tollef Fog Heen <tfheen@xxxxxxxxxx>
   varnish (U)

Tomasz Buchert <tomasz@xxxxxxxxxx>
   miredo

Varnish Package Maintainers <pkg-varnish-devel@xxxxxxxxxxxxxxxxxxxxxxx>
   varnish