Skip to content
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

1.27.0: not ready for gettext 0.22 #390

Open
kloczek opened this issue Aug 23, 2023 · 16 comments
Open

1.27.0: not ready for gettext 0.22 #390

kloczek opened this issue Aug 23, 2023 · 16 comments
Labels
template ignored! Basic informations are missing

Comments

@kloczek
Copy link

kloczek commented Aug 23, 2023

rm -f hu.gmo && /usr/bin/msgmerge --for-msgfmt -o hu.1po hu.po mate-power-manager.pot && /usr/bin/msgfmt -c --statistics --verbose -o hu.gmo hu.1po && rm -f hu.1po
hu.1po:980: 'msgstr' is not a valid C format string, unlike 'msgid'. Reason: In the directive number 2, the argument size specifier is invalid.
/usr/bin/msgfmt: found 1 fatal error
hu.1po: 380 translated messages.
@raveit65
Copy link
Member

@raveit65 raveit65 added the template ignored! Basic informations are missing label Aug 23, 2023
@kloczek
Copy link
Author

kloczek commented Aug 23, 2023

Hmm 🤔 .. that link has nothing to do with reported issue.

@kloczek
Copy link
Author

kloczek commented Sep 28, 2023

gentle ping .. 🤔

@raveit65
Copy link
Member

This is an translation error caused by translators at transifex.
For that reason i gave you the link to transifex.
This have to be fixed there.

@kloczek
Copy link
Author

kloczek commented Oct 15, 2023

On that link are only statistics of the translations.

@raveit65
Copy link
Member

You need a account, become a member of your language team and translate.
Sorry i can't help you more.

@kloczek
Copy link
Author

kloczek commented Oct 15, 2023

Why I need such help?
Broken file is in this git repo.
If fix is already present at transifex why this cannot be commited? 🤔

@raveit65
Copy link
Member

Is the fix already present at transifex?
Why do you think that?
Btw. we don't have this issue in CI.

@kloczek
Copy link
Author

kloczek commented Oct 15, 2023

You pointed on transifex as place where I can find solution.
I'm not thinking about anything.

@kloczek
Copy link
Author

kloczek commented Oct 15, 2023

Btw. we don't have this issue in CI.

Again .. I;'m using gettext 0.22.

@raveit65
Copy link
Member

raveit65 commented Oct 16, 2023

You pointed on transifex as place where I can find solution.

No, you should become a translator of your language and fix it for yourself.
I can't do that for you because I don't speak Hungary language

@kloczek
Copy link
Author

kloczek commented Oct 16, 2023

This about HUNGARIAN translation.
I'm Polish and I'm not translator.
Fact is that there is no fixes in git repo.

@paravoid
Copy link

paravoid commented Aug 8, 2024

This is breaking the build on recent Debian systems, see https://bugs.debian.org/1074753. In fact, it has resulted in mate-power-manager being removed from testing/trixie.

I'm not a Hungarian speaker either, but the issue is with the format specifier, not the text: replacing %.0lf%% with %.0f%% (like the original English text, and every other translation) fixes this.

@lukefromdc
Copy link
Member

lukefromdc commented Aug 9, 2024 via email

@paravoid
Copy link

I understand that there is probably a wider translation maintainership issue to consider here. I'm not too familiar with the translation issues you're mentioning but I trust that you know best :)

That said, I wouldn't let perfect be the enemy of good here: we're talking about an obvious single character patch that would immediately fix this build failure. I'd commit that, and then perhaps continue discussing how to handle broken translations?

@lukefromdc
Copy link
Member

lukefromdc commented Aug 14, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
template ignored! Basic informations are missing
Projects
None yet
Development

No branches or pull requests

4 participants