Attachment 133748 Details for Bug 102388 – 0002-stderr.patch

[patch] 0002-stderr.patch

0002-stderr.patch (text/plain), 5.46 KB, created by Laurent Bigonville on 2017-08-24 14:22:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Laurent Bigonville

Created: 2017-08-24 14:22:46 UTC

Size: 5.46 KB

patch

obsolete

>Author: Daniel Baumann <mail@daniel-baumann.ch>
>Description:
> Use fd for stdout instead of device, since the latter may not always exist,
> e.g. in chroots (Closes: #581649).
>
>--- a/scripts/plymouth-populate-initrd.in
>+++ b/scripts/plymouth-populate-initrd.in
>@@ -340,7 +340,7 @@ function usage() {
>     local output="/dev/stdout"
>     local rc=0
>     if [ "$1" == "error" ]; then
>-        output="/dev/stderr"
>+        output=">&2"
>         rc=1
>     fi
> 
>@@ -385,13 +385,13 @@ inst ${PLYMOUTH_POLICYDIR}/plymouthd.def
> inst ${PLYMOUTH_CONFDIR}/plymouthd.conf $INITRDDIR
> 
> if [ -z "$PLYMOUTH_THEME_NAME" ]; then
>-    echo "No default plymouth plugin is set" > /dev/stderr
>+    echo "No default plymouth plugin is set" >&2
>     exit 1
> fi
> 
> if [ $THEME_OVERRIDE ]; then
>     conf=$INITRDDIR/${PLYMOUTH_CONFDIR}/plymouthd.conf
>-    echo "modifying plymouthd.conf: Theme=$PLYMOUTH_THEME_NAME" > /dev/stderr
>+    echo "modifying plymouthd.conf: Theme=$PLYMOUTH_THEME_NAME" >&2
>     # make sure the section and key exist so we can modify them
>     grep -q "^ *\[Daemon\]" $conf || echo "[Daemon]" >> $conf
>     grep -q "^ *Theme *=" $conf || echo "Theme=fade-in" >> $conf
>@@ -401,7 +401,7 @@ fi
> PLYMOUTH_MODULE_NAME=$(grep "ModuleName *= *" ${PLYMOUTH_DATADIR}/plymouth/themes/${PLYMOUTH_THEME_NAME}/${PLYMOUTH_THEME_NAME}.plymouth | sed 's/ModuleName *= *//')
> 
> if [ ! -f ${PLYMOUTH_PLUGIN_PATH}/${PLYMOUTH_MODULE_NAME}.so ]; then
>-    echo "The default plymouth plugin (${PLYMOUTH_MODULE_NAME}) doesn't exist" > /dev/stderr
>+    echo "The default plymouth plugin (${PLYMOUTH_MODULE_NAME}) doesn't exist" >&2
>     exit 1
> fi
> 
>--- a/scripts/plymouth-set-default-theme.in
>+++ b/scripts/plymouth-set-default-theme.in
>@@ -76,14 +76,14 @@ while [ $# -gt 0 ]; do
> 
>         -l|--list)
>                 if [ -n "$THEME_NAME" ]; then
>-                        echo "You can only specify --list or a theme name, not both" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify --list or a theme name, not both" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>                 if [ $DO_RESET -ne 0 ]; then
>-                        echo "You can only specify --reset or --list, not both" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify --reset or --list, not both" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>@@ -96,14 +96,14 @@ while [ $# -gt 0 ]; do
> 
>         -r|--reset|default)
>                 if [ -n "$THEME_NAME" ]; then
>-                        echo "You can only specify --reset or a theme name, not both" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify --reset or a theme name, not both" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>                 if [ $DO_LIST -ne 0 ]; then
>-                        echo "You can only specify --reset or --list, not both" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify --reset or --list, not both" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>@@ -116,20 +116,20 @@ while [ $# -gt 0 ]; do
> 
>         *)
>                 if [ -n "$THEME_NAME" ]; then
>-                        echo "You can only specify one theme at a time" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify one theme at a time" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>                 if [ $DO_RESET -ne 0 ]; then
>-                        echo "You can only specify --reset or a theme name, not both" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify --reset or a theme name, not both" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>                 if [ $DO_LIST -ne 0 ]; then
>-                        echo "You can only specify --list or a theme name, not both" > /dev/stderr
>-                        echo $(usage) > /dev/stderr
>+                        echo "You can only specify --list or a theme name, not both" >&2
>+                        echo $(usage) >&2
>                         exit 1
>                 fi
> 
>@@ -155,7 +155,7 @@ if [ $DO_RESET -eq 0 ] && [ $DO_INITRD_R
> fi
> 
> if [ `id -u` -ne 0 ]; then
>-        echo "This program must be run as root" > /dev/stderr
>+        echo "This program must be run as root" >&2
>         exit 1
> fi
> 
>@@ -166,14 +166,14 @@ if [ $DO_RESET -ne 0 ]; then
> fi
> 
> if [ ! -e ${PLYMOUTH_DATADIR}/plymouth/themes/${THEME_NAME}/${THEME_NAME}.plymouth ]; then
>-        echo "${PLYMOUTH_DATADIR}/plymouth/themes/${THEME_NAME}/${THEME_NAME}.plymouth does not exist" > /dev/stderr
>+        echo "${PLYMOUTH_DATADIR}/plymouth/themes/${THEME_NAME}/${THEME_NAME}.plymouth does not exist" >&2
>         exit 1
> fi
> 
> MODULE_NAME=$(grep "ModuleName *= *" ${PLYMOUTH_DATADIR}/plymouth/themes/${THEME_NAME}/${THEME_NAME}.plymouth | sed 's/ModuleName *= *//')
> 
> if [ ! -e ${PLYMOUTH_PLUGIN_PATH}${MODULE_NAME}.so ]; then
>-        echo "${PLYMOUTH_PLUGIN_PATH}${MODULE_NAME}.so does not exist" > /dev/stderr
>+        echo "${PLYMOUTH_PLUGIN_PATH}${MODULE_NAME}.so does not exist" >&2
>         exit 1
> fi
>

Actions: View | Splinter Review

Attachments on bug 102388: 133748 | 133753

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.