[libvirt PATCH 00/51] Use permutable format strings in translations
Jiri Denemark
jdenemar at redhat.com
Thu Mar 30 09:37:55 UTC 2023
On Mon, Mar 27, 2023 at 15:37:34 +0100, Daniel P. Berrangé wrote:
> On Mon, Mar 27, 2023 at 01:08:09PM +0200, Jiri Denemark wrote:
> > On Fri, Mar 10, 2023 at 17:14:32 +0000, Daniel P. Berrangé wrote:
> > > Even if fixed, it might be worth switching the .pot file anyway, but
> > > this can't be done without us bulk updating the translations, and
> > > bulk re-importing them, which will be challenging. We'll almost
> > > certainly want to try this on a throw-away repo in weblate first,
> > > not our main repo.
> >
> > I was able to come up with steps leading to the desired state:
> >
> > 0. lock weblate repository
> > 1. update libvirt.pot from the most recent potfile job
> > 2. push to libvirt.git
> > 2. wait for translations update from Fedora Weblate and merge it
> > 3. pull from libvirt.git
> > 4. apply the first 50 patches from this seires (with required changes
> > to make sure all translation strings are updated)
> > 5. update all po files with the attached script
> > 6. update libvirt.pot by running meson compile libvirt-pot
> > 7. apply patch 51 of this series
> > 8. push to libvirt.git
> > 9. wait for translations update from Fedora Weblate and merge it
> > 10. unlock weblate repository
> >
> > The process takes about an hour if we're lucky as weblate is quite slow
> > when processing such large amount of changes.
> >
> > The result can be seen at
> >
> > https://gitlab.com/jirkade/libvirt/-/commits/format-strings
> >
> > and the corresponding weblate repository at
> >
> > https://translate.fedoraproject.org/projects/libvirt/test/
> >
> > I used d05ad0f15e737fa2327dd68870a485821505b58f commit as a base.
>
> Looking at this, I picked a random language (Bengali) and compared
> stats:
>
> https://translate.fedoraproject.org/projects/libvirt/test/bn_IN/
>
> vs
>
> https://translate.fedoraproject.org/projects/libvirt/libvirt/bn_IN/
>
> Translated strings matches to within 2 words, which is probably
> accounted for by being based on different HEAD
>
> Strings with failing checks is massively different, and that is
> the fault of 'failing check: C format' - 1300 more failing checks
> afterwards.
Oops, my random generator apparently selected wrong language where the
number of 'failing check: C format' issues was significantly lower then
before :-)
Anyway, I did what you suggested and updated the repositories
https://gitlab.com/jirkade/libvirt/-/commits/format-strings
https://translate.fedoraproject.org/projects/libvirt/test/
The content is now based on v9.2.0-rc1-8-geb677e3a10 which is just one
commit behind 9.2.0-rc2.
The fixes to whitespace and cpp string concatenation were sent
separately to the list earlier today.
Jirka
More information about the libvir-list
mailing list