It's necessary when doing if test -n <xsl:value-of ...> as `if test -n `
will simply return true.
For those `if test -z ` it's not strictly necessary, but doesn't hurt
having it there.
Signed-off-by: Fabiano Fidêncio <fabiano@xxxxxxxxxxxx>
---
data/install-script/redhat.com/rhel-kickstart-desktop.xml.in | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/data/install-script/redhat.com/rhel-kickstart-desktop.xml.in b/data/install-script/redhat.com/rhel-kickstart-desktop.xml.in
index 336c746..e14a67b 100644
--- a/data/install-script/redhat.com/rhel-kickstart-desktop.xml.in
+++ b/data/install-script/redhat.com/rhel-kickstart-desktop.xml.in
@@ -118,20 +118,20 @@ qemu-guest-agent
%post --erroronfail
useradd -G wheel <xsl:value-of select="config/user-login"/> # Add user
-if test -z <xsl:value-of select="config/user-password"/>; then
+if test -z "<xsl:value-of select="config/user-password"/>"; then
passwd -d <xsl:value-of select="config/user-login"/> # Make user account passwordless
else
echo <xsl:value-of select="config/user-password"/> |passwd --stdin <xsl:value-of select="config/user-login"/>
fi
-if test -z <xsl:value-of select="config/admin-password"/>; then
+if test -z "<xsl:value-of select="config/admin-password"/>"; then
passwd -d root # Make root account passwordless
else
echo <xsl:value-of select="config/admin-password"/> |passwd --stdin root
fi
# Set user avatar
-if test -n <xsl:value-of select="config/avatar-location"/>; then
+if test -n "<xsl:value-of select="config/avatar-location"/>"; then
mkdir /mnt/unattended-media
mount <xsl:value-of select='config/avatar-disk'/> /mnt/unattended-media
cp /mnt/unattended-media<xsl:value-of select="config/avatar-location"/> /home/<xsl:value-of select="config/user-login"/>/.face
--
2.9.3
_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo
