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>
---
.../fedoraproject.org/fedora-kickstart-desktop.xml.in | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in b/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
index 1fefebb..58f8d93 100644
--- a/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
+++ b/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
@@ -173,20 +173,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"/> /var/lib/AccountsService/icons/<xsl:value-of select="config/user-login"/>
--
2.9.3
_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo
