[Freeipa-devel] [PATCH] admiyo-freeipa-0044-I18N-for-web.patch

[Adam Young]

On 09/27/2010 10:16 PM, Rob Crittenden wrote:
> Adam Young wrote:
>> On 09/27/2010 12:45 PM, Adam Young wrote:
>>> On 09/27/2010 10:35 AM, Adam Young wrote:
>>>> On 09/24/2010 09:43 PM, Adam Young wrote:
>>>>> I18N for web
>>>>>
>>>>> Performing I18N completely on the server, to leverage the existing
>>>>> gettext architecture.
>>>>> Also, the browser does not have access to the Language header.
>>>>>
>>>>> Added the additional po files for a set of required languages
>>>>>
>>>>> conflict with install/static/ipa.js was resolved.
>>>>>
>>>>>
>>>>>
>>>>> Note that the addition of the .po files in this patch is necessary.
>>>>> In order to get Transifex support, we need to update the LINGUAS
>>>>> file with the languages for which we want support.
>>>>>
>>>>> If we don't add the .po files in, they get automatically generated
>>>>> by the rpmbuild process.
>>>>>
>>>>> Our implementation of gettext has a bug in it (It might be F13
>>>>> thing) where the the Plurals line is not getting correctly
>>>>> transformed, which causes a build failure. However, since the RPM
>>>>> would have the .po files anyway, we should revision control the ones
>>>>> we have, even if they are empty.
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Freeipa-devel mailing list
>>>>> Freeipa-devel at redhat.com
>>>>> https://www.redhat.com/mailman/listinfo/freeipa-devel
>>>> Returned the Bug reporting URL to its origianl value. Looks like
>>>> another bug in the gettext mechanims.
>>>>
>>>>
>>>> _______________________________________________
>>>> Freeipa-devel mailing list
>>>> Freeipa-devel at redhat.com
>>>> https://www.redhat.com/mailman/listinfo/freeipa-devel
>>> Corrected 'Plurals' lines in regenerated PO files.
>>>
>>>
>>> _______________________________________________
>>> Freeipa-devel mailing list
>>> Freeipa-devel at redhat.com
>>> https://www.redhat.com/mailman/listinfo/freeipa-devel
>> ACKed in IRC by edewata and pushed to master
>
> This patch includes install/po/Makefile which is supposed to be an 
> auto-generated file. Can you remove it?
>
> rob
Will do