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

Fabiano Fidêncio fabiano at fidencio.org
Wed May 4 16:30:29 UTC 2016


On Wed, May 4, 2016 at 6:24 PM, Zeeshan Ali (Khattak)
<zeeshanak at gnome.org> wrote:
> On Tue, May 3, 2016 at 7:05 PM, Fabiano Fidêncio <fidencio at redhat.com> 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".

Adding to the list (as a reminder):
* the hack the hack -> the hack
* Windws -> Windows

I'll fix the grammatical mistakes and push the series later on Today.

>
> Feel free to fix these before pushing.
>
>> Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
>> ---
>>  .../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 at redhat.com
>> https://www.redhat.com/mailman/listinfo/libosinfo
>
>
>
> --
> Regards,
>
> Zeeshan Ali (Khattak)
>
> _______________________________________________
> Libosinfo mailing list
> Libosinfo at redhat.com
> https://www.redhat.com/mailman/listinfo/libosinfo



-- 
Fabiano Fidêncio




More information about the Libosinfo mailing list