Bug 432970

Summary: Wrong Release Notes format (in openSUSE)
Product: [openSUSE] openSUSE 11.1 Reporter: Karl Eichwalder <ke>
Component: YaST2Assignee: Karl Eichwalder <ke>
Status: RESOLVED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Major    
Priority: P2 - High CC: birger.kollstrand, gs, kukuk, locilka
Version: Beta 2   
Target Milestone: Beta 4   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: Documentation Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: y2log

Description Karl Eichwalder 2008-10-07 10:33:21 UTC
If you install with textmode=1 (yast ncurses) using English, yast tries to display the release notes of all languages, but for example, fails to display Czech ů-letters and all the Russian letters properly.

At least, try to preselect English as the default (that's what I also like to recommend for the GUI).
Comment 1 Katarina Machalkova 2008-10-07 12:24:42 UTC
I'm afraid I don't quite understand. If the language is English, the english version of release notes is shown and there are obviously no Czech/Russian letters in it, or ... ?

Or maybe I have different dialog in mind, can you please specify how to reproduce the bug (i.e. how to have yast display R.N. in all languages)?
Comment 2 Karl Eichwalder 2008-10-07 13:03:40 UTC
Created attachment 243948 [details]
y2log

After rebooting, at the end of the installation once the Internet test is done.  Because "cs" comes first in openSUSE, yast displays it.
Comment 3 Gabriele Mohr 2008-10-16 11:21:20 UTC
inst_release_notes.ycp fails to find the preferred language because the regexpmatch doesn't work any longer. The file name obviously has changed (was RELEASE-NOTES.<lang>.rtf and now is RELEASE-NOTES-<product>.<lang>.rtf).
There are also some other places where the regular expression doesn't fit.

Proposed patch:

Index: src/clients/inst_release_notes.ycp
===================================================================
128c128
<           return regexpmatch (one_relnotes, "^RELEASE-NOTES\..*\.rtf$");
---
>           return regexpmatch (one_relnotes, "^RELEASE-NOTES.*\\..*\\.rtf$");
133c133
<           string relnotes_lang = regexpsub (one_relnotes, "^RELEASE-NOTES\.(.*)\.rtf$", "\\1");
---
>           string relnotes_lang = regexpsub (one_relnotes, "^RELEASE-NOTES.*\\.(.*)\\.rtf$", "\\1");
156c156
<               if (regexpmatch (item_id, sformat ("RELEASE-NOTES\.%1\.rtf$", preffered_lang))) {
---
>               if (regexpmatch (item_id, sformat ("RELEASE-NOTES.*\\.%1\\.rtf$", preffered_lang))) {


The '.' has to be escaped with '\\'.

I can't think of a solution for the other problem that after changing the language e.g. to czech some characters aren't displayed correctly. To display the characters of the choosen language, the appropriate font has to be loaded. But the button labels and text will remain in installation language.
Comment 5 Karl Eichwalder 2008-10-17 13:31:15 UTC
Submitted to /work.
Comment 7 Karl Eichwalder 2008-10-24 14:04:21 UTC
*** Bug 438112 has been marked as a duplicate of this bug. ***
Comment 8 Karl Eichwalder 2008-10-27 08:21:27 UTC
*** Bug 439011 has been marked as a duplicate of this bug. ***