-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add download for rpm-groups.pot #28
Conversation
👍 |
As I understand correctly, a new rpm-groups.pot generator was added to packages-i18n. Correct rpm-groups.pot is saved there, and here we want to import it and translate. It is surely possible. We just have to remember that any we should never generate rpm-groups po files in packages-i18n (and not import them to Weblate). There is another possibility: migrate rpm-groups to packages-i18n completely. But as po files there are used for a different purpose than installation of mo files, it would complicate the process. So I agree. The new script in yast-translations could be easily called on a regular basis. |
The new script has a different string order. I did the first call and compared results. The new pot contains the same string set, just in a different order. There are only two exceptions, both are bugs. @lnussel Please fix: Once it will be done, I will perform the call. It will generate a large diff, but no real changes, and no work for translators. |
Don't worry the old script wasn't called automatically either and was broken anyways. So should be ok to just track updating rpm-groups.pot in the project plan. It's not like rpm groups change often anyways :-) Wrt to "sources" and "Gui". If the string appears in there I suppose there are packages that use those strings. @m-rey can you check which packages and fix them if it's wrong indeed? |
I've tweaked the packages-i18n script to also output the package name in the comment line. Apparently, there are some packages that use "Gui" in the RPM group:
And here's the package using "sources":
Also, there is a package using "Yast" instead of "YaST":
|
there are actually tons of packages with invalid groups: https://rpmlint.opensuse.org/openSUSE:Factory/x86_64/standard?rule=non-standard-group so spelling mistakes is just one kind of error |
Fixed: I will manually edit the pot before commit, so we don't have to wait for check-in. |
With this PR, an updated version of rpm-groups.pot from the https://github.com/openSUSE/packages-i18n repository is downloaded and used.