On Thu, Jan 16, 2020 at 1:26 PM Daniel P. Berrangé <berrange(a)redhat.com> wrote:
On Thu, Jan 16, 2020 at 01:17:18PM +0100, Jean-Baptiste Holcroft wrote:
> Hi,
>
> Le 2020-01-16 12:52, Fabiano Fidêncio a écrit :
> > Firstly, we're receiving MRs on our GitLab directly changing po/*po
> > files. is that expected?
>
> Yes, your component uses the plugin "Update PO files to match POT
> (msgmerge)"
>
https://docs.weblate.org/en/weblate-3.10.1/admin/addons.html#update-po-fi...
>
> This will update automatically all po files, allowing translators to see
> missing translations without having you to care about updating all files.
>
> > If so, what would be the best way to:
> > 1) Avoid things like
> >
https://translate.stg.fedoraproject.org/projects/libosinfo/libosinfo/#ale...
>
> Here you have the Polish file in conflicts. It probably means it was edited
> in both git repositories.
I don't believe that's the case here if I look at the git commit history
In the main upstream libosinfo.git repo we have this commit:
commit 3d6422ced93c3f7797300bd8476a5ca9ab492217
Author: Piotr Drąg <piotrdrag(a)gmail.com>
AuthorDate: Sat Jan 11 11:03:47 2020 +0000
Commit: Fabiano Fidêncio <fidencio(a)redhat.com>
CommitDate: Mon Jan 13 08:22:15 2020 +0100
Translated using Weblate (Polish)
Currently translated at 100.0% (168 of 168 strings)
Translation: libosinfo/libosinfo
Translate-URL:
https://translate.stg.fedoraproject.org/projects/libosinfo/libosinfo/pl/
Signed-off-by: Piotr Drąg <piotrdrag(a)gmail.com>
This was from a Weblate merge request merged on Monday:
https://gitlab.com/libosinfo/libosinfo/merge_requests/43
If we look at the Weblate git repo though, the commit was
updated by the translator again on Wednesday, *after* the
weblate merge request was already accepted:
commit 17e6dde789283689a36dafe2bd9c453d671b6ec3
Author: Piotr Drąg <piotrdrag(a)gmail.com>
Date: Wed Jan 15 15:25:28 2020 +0000
Translated using Weblate (Polish)
Currently translated at 100.0% (168 of 168 strings)
Translation: libosinfo/libosinfo
Translate-URL:
https://translate.stg.fedoraproject.org/projects/libosinfo/libosinfo/pl/
Signed-off-by: Piotr Drąg <piotrdrag(a)gmail.com>
So AFAICT, the cause of the conflict is weblate itself.
It should never have amended the existing commit because it
had already been accepted via the merge request 2 days earlier.
For some reason weblate did not see that the merge request was
accepted and so did not refresh its view of the upstream repo,
allowing the translator to continue working on outdated content
ultimately resulting in this conflict.
For now, libosinfo repo is locked. Jean-Baptiste, would you mind
giving me some hands-on help on solving this?
We could do this via IRC, or any other communication platform.
Also, osinfo-db-tools will face similar issues because, for some
reason I don't understand (as everything seems to be correctly set-up)
may be in a similar state if I decide to close this MR[0] and just
push the changes to master adding my own SoB tag.
[0]: