[osinfo-db PATCH 1/5] install-script, windows, desktop: Do not set user avatar for win8+

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



For Windows 8+ the image format and expected sizes of the image are
different than what's used for Windows 7 (or older).

This could be implemented (and then, re-enabled) in the future. But, for
now, let's just not set user avatar for win8+ as it's not going to work
there with the current state of libosinfo & consumers.

Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
---
 .../install-script/microsoft.com/windows-cmd-desktop.xml.in | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/data/install-script/microsoft.com/windows-cmd-desktop.xml.in b/data/install-script/microsoft.com/windows-cmd-desktop.xml.in
index 7fee340..978bafd 100644
--- a/data/install-script/microsoft.com/windows-cmd-desktop.xml.in
+++ b/data/install-script/microsoft.com/windows-cmd-desktop.xml.in
@@ -86,8 +86,14 @@ net localgroup administrators <xsl:value-of select="config/user-realname"/> /add
 net accounts /maxpwage:unlimited
 REGEDIT /S <xsl:call-template name="script-disk"/>:\windows.reg
 </xsl:if>
+<!-- Do not try to copy the user avatar for Windows 8+.
+     Windows 8+ needs a more than one image, with different sizes, in order to
+     properly set the user's avatar all over the place.
+     Reference: https://gitlab.com/libosinfo/osinfo-db/issues/24 -->
+<xsl:if test="os/version &lt; 6.2">
 if not "<xsl:value-of select="config/avatar-location"/>"=="" copy "<xsl:value-of select="config/avatar-disk"/>:<xsl:value-of select="config/avatar-location"/>" "<xsl:call-template name="avatar-destination"/>
 <xsl:text>&#xa;</xsl:text>
+</xsl:if>
 
 <xsl:call-template name="post-install-drivers-disk"/>:
 <xsl:if test="config/driver-signing = 'false' and os/version &gt; 5.1">
-- 
2.21.0

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux