Re: [PATCH v5 12/14] installer: Use l10n-language as UI language

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

 



On Tue, May 3, 2016 at 7:05 PM, Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote:
> Since commit 57b459e09 we set 'EN-US' as UI language instead of 'EN-GB'.
> However, this hack is only necessary for Windows 7 and can cause issues
> when installing newer versions of the Windows, as the installer,
> somehow, gets confused by having different values set to UI language,
> System Locale and User Locale and ends up prompting a window for letting
> the user to set these values.
> Simply by limiting the hack the hack to Windws 7 fixes the installation
> of 'EN-GB' ISOs of Windows 8 and 8.1 (the ones tested for now).

Grammatical mistakes:

* Empty line before each para, please.
* comma missing before "fixes" here?
* Subject missing in last sentence, "fixes" -> "this patch fixes".

Feel free to fix these before pushing.

> Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> ---
>  .../microsoft.com/windows-unattend-desktop.xml.in      | 18 ++++++++++++++++--
>  1 file changed, 16 insertions(+), 2 deletions(-)
>
> diff --git a/data/install-script/microsoft.com/windows-unattend-desktop.xml.in b/data/install-script/microsoft.com/windows-unattend-desktop.xml.in
> index 0bc1aeb..310246f 100644
> --- a/data/install-script/microsoft.com/windows-unattend-desktop.xml.in
> +++ b/data/install-script/microsoft.com/windows-unattend-desktop.xml.in
> @@ -170,10 +170,24 @@
>            <xsl:call-template name="arch"/>
>           </xsl:attribute>
>           <SetupUILanguage>
> -          <UILanguage><xsl:call-template name="ui-language"/></UILanguage>
> +          <xsl:choose>
> +           <xsl:when test="os/version = 6.1">
> +            <UILanguage><xsl:call-template name="ui-language"/></UILanguage>
> +           </xsl:when>
> +           <xsl:otherwise>
> +            <UILanguage><xsl:value-of select="config/l10n-language"/></UILanguage>
> +           </xsl:otherwise>
> +          </xsl:choose>
>           </SetupUILanguage>
>           <SystemLocale><xsl:value-of select="config/l10n-language"/></SystemLocale>
> -         <UILanguage><xsl:call-template name="ui-language"/></UILanguage>
> +         <xsl:choose>
> +          <xsl:when test="os/version = 6.1">
> +           <UILanguage><xsl:call-template name="ui-language"/></UILanguage>
> +          </xsl:when>
> +          <xsl:otherwise>
> +           <UILanguage><xsl:value-of select="config/l10n-language"/></UILanguage>
> +          </xsl:otherwise>
> +         </xsl:choose>
>           <UserLocale><xsl:value-of select="config/l10n-language"/></UserLocale>
>          </component>
>         </settings>
> --
> 2.7.3
>
> _______________________________________________
> Libosinfo mailing list
> Libosinfo@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libosinfo



-- 
Regards,

Zeeshan Ali (Khattak)

_______________________________________________
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