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
_______________________________________________
pkg-fso-maint mailing list
pkg-fso-maint@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-fso-maint

Reply via email to