Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0044-I18N-for-web.patch
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@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@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Corrected 'Plurals' lines in regenerated PO files. ___ Freeipa-devel mailing list Freeipa-devel@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 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0044-I18N-for-web.patch
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@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@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel Corrected 'Plurals' lines in regenerated PO files. ___ Freeipa-devel mailing list Freeipa-devel@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 ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel