|
Bugzilla – Full Text Bug Listing |
| Summary: | Yast fonts module crashes to open | ||
|---|---|---|---|
| Product: | [openSUSE] openSUSE Distribution | Reporter: | Forgotten User cNsFFHOGnn <forgotten_cNsFFHOGnn> |
| Component: | YaST2 | Assignee: | Karl Eichwalder <ke> |
| Status: | RESOLVED WONTFIX | QA Contact: | Jiri Srain <jsrain> |
| Severity: | Normal | ||
| Priority: | P5 - None | CC: | elchevive68, forgotten_cNsFFHOGnn, lnussel, mpluskal, pgajdos, sbrabec, slemke |
| Version: | Leap 42.2 | ||
| Target Milestone: | --- | ||
| Hardware: | x86-64 | ||
| OS: | openSUSE 42.2 | ||
| Whiteboard: | |||
| Found By: | --- | Services Priority: | |
| Business Priority: | Blocker: | --- | |
| Marketing QA Status: | --- | IT Deployment: | --- |
| Attachments: | YaST log file | ||
|
Description
Forgotten User cNsFFHOGnn
2017-01-06 15:57:47 UTC
Created attachment 708928 [details]
YaST log file
I think I have work for cca two or three months currently on my table, no time for this one. Just a quick look. It does not on my 42.2 fully patched system. What version of yast2-fonts do you have installed? I have ruby2.1-rubygem-fast_gettext-0.9.2-6.3.x86_64 installed on my system, you seem to have higher version, not sure if that could case troubles. Versions installed: yast2-fonts: 3.1.17-4.1 ruby2.1-rubygem-fast_gettext-1_2: 1.2.0-1.1 ruby2.1-rubygem-fast_gettext: Not installed ruby2.1-rubygem-fast_gettext-1_1: Not installed Needinfo was provided. Hi, I see that the language used is pt_BR. I found an error in a translation which probably is the cause of this problem: <p><b>Specimen for %s</b></p> -> <p><b>Espécie de %1 </b></p> I fixed the wrong "%1" in Weblate: https://l10n.opensuse.org/translate/yast-fonts/master/pt_BR/?sid=a0172bf4-f586-11e6-8772-525400a48651&offset=0 The same applies to bug 1021982 (https://bugzilla.opensuse.org/show_bug.cgi?id=1021982) Hello, I did and ugly 'on the fly' test and indeed this change fixes the problem: Edited: vim /usr/share/YaST2/locale/pt_BR/LC_MESSAGES/fonts.mo Changed: <p><b>Espécie de %1 </b></p> to <p><b>Espécie de %s </b></p> So that translation update fixes the segmentation fault. *** Bug 1021982 has been marked as a duplicate of this bug. *** Ah, learning occurred on my side. Thank you for digging in! Hi, I'm wondering why Weblate didn't find this errors, as "%s" is a common variable in YaST, maybe we should add this check also. I'll open a bug about it. Regards Karl, could you please decide if we will do a maintenance update for yast2-trans [workaround exists]. (In reply to Petr Gajdos from comment #12) > could you please decide if we will do a maintenance update for yast2-trans > [workaround exists]. If the workaround will prevent yast from crashing, we should do the maintenance update. Of course, the workaround must be save, not introducing unrelated problems, etc. pp. As I understand it, the fix from comment 8 is a real fix. By the workaround I mean 'using yast2 fonts under another locale'. You seem to maintain yast2-trans, that's why I leave decision to you. Adding Stanislav to CC. It is possible to create an update with just a single fix. Or we can do a best attempt merge with the latest contents of Weblate, which possibly fixes some issues, but it also could possibly introduce some issues. Comment 11: Weblate did not found the problem because pot file does not provide format information. Without that, "$1" or "%s" are considered as an ordinary sequence of characters. See bug 980329 for details. Hi, Couldn't we put some kind of quality checks for %s while Ruby is not supported by gettext? https://docs.weblate.org/en/latest/admin/checks.html#writing-own-checks (In reply to Petr Gajdos from comment #14) > As I understand it, the fix from comment 8 is a real fix. By the workaround > I mean 'using yast2 fonts under another locale'. You seem to maintain > yast2-trans, that's why I leave decision to you. A package update would be ok for me. Unfortunately, there does not seem to be an easy command to rebuild a suitable .tar with the fixed translation. We now have a _service and are using git. As long as nobody is able to tell the exact command sequence needed, I cannot submit the package update. Sad. Reopening as per discussion off-line. I've created a branch "openSUSE-42_2" in the yast2-translations repo. I think the next step would be to add this branch as option to all yast modules in weblate. Then language translators can submit translations for 42.2 maintenance updates. Stanislav could you configure that please? Karl, in the _service file you need '<param name="revision">openSUSE-42_2</param>' OK, I can add yast-translations openSUSE-42_2 to Weblate. It will mean adding of 97 new components. I will use a script for that. This is automated batch bugzilla cleanup. The openSUSE 42.2 changed to end-of-life (EOL [1]) status. As such it is no longer maintained, which means that it will not receive any further security or bug fix updates. As a result we are closing this bug. If you can reproduce this bug against a currently maintained version of openSUSE, or you can still observe it under openSUSE Leap 15.0, please feel free to reopen this bug against that version (see the "Version" component in the bug fields), or alternatively open a new ticket. Thank you for reporting this bug and we are sorry it could not be fixed during the lifetime of the release. [1] https://en.opensuse.org/Lifetime |