Package: nodm
Version: 0.13-1
Severity: normal
Tags: patch
Dear maintainer,
nodm.postinst should not write to stdout, since messages written to
stdout may confuse debconf. See the following snippnet from
debconf-devel(7) man page[1]:
Avoid outputting anything to stdout in your postinst, since that can
confuse debconf, and postinst should not be verbose anyway. Output
to stderr is ok, if you must.
[1] https://manpages.debian.org/unstable/debconf-doc/debconf-devel.7.en.html
Best,
--
Ilias
diff -ru old/debian/nodm.postinst new/debian/nodm.postinst
--- old/debian/nodm.postinst 2017-05-02 23:57:24.000000000 +0300
+++ new/debian/nodm.postinst 2017-05-05 16:33:31.874949860 +0300
@@ -18,7 +18,7 @@
RET="$THIS_PACKAGE"
fi
if [ "$THIS_PACKAGE" != "$RET" ]; then
- echo "Please be sure to run \"dpkg --configure $RET\"."
+ echo "Please be sure to run \"dpkg --configure $RET\"." >&2
fi
if db_get "$RET"/daemon_name; then
echo "$RET" > $DEFAULT_DISPLAY_MANAGER_FILE
