Bug 10942: (QA followup) make warnings to to STDERR
authorTomas Cohen Arazi <tomascohen@gmail.com>
Thu, 13 Mar 2014 14:30:15 +0000 (11:30 -0300)
committerGalen Charlton <gmc@esilibrary.com>
Thu, 24 Apr 2014 12:50:54 +0000 (12:50 +0000)
As noted by Robin, STDOUT is used by the script to communicate with
debconf and, hence, the warning messages should be directed to STDERR.

This patch does that. To test:

- Set AUTOMATIC_TRANSLATIONS_UPDATE="no" so the warning normally shows
- Redirect STDOUT to /dev/null:
  dpkg -i koha-common...deb > /dev/null
=> Warning message doesn't show (i.e. it is sent to STDOUT)
- Apply the patch, rebuild package
- Redirect STDOUT to /dev/null:
  dpkg -i koha-common...deb > /dev/null
=> Warning message shows (i.e. is correctly sent to STDERR)
- Redirect STDERR to /dev/null:
  dpkg -i koha-common...deb 2> /dev/null
=> Warning message doesn't show
- Verify that previous tests pass

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
debian/koha-common.postinst

index 47acd42..f22bcb0 100644 (file)
@@ -59,7 +59,7 @@ if [ "$AUTOMATIC_TRANSLATIONS_UPDATE" = "yes" ]; then
         if koha-translate --update $lang; then
             echo "Updated the $lang translations."
         else
-            cat <<EOF
+            cat <<EOF >&2
 ERROR: an error was found when updating '$lang' translations. Please manually
 run 'koha-translate --update $lang'. Run man koha-translate for more options.
 EOF
@@ -69,7 +69,7 @@ else
     # no auto-update, check update needed and warn if needed
     if koha-translate --list | grep -v -q -x "en"; then
         # translations installed, update needed
-        cat <<EOF
+        cat <<EOF >&2
 Warning: template translations are not set to be automatically updated.
 Please manually run 'koha-translate --update lang_code' to update them.