Red Hat Bugzilla – Bug 622064
Support translations in product.img installclasses
Last modified: 2010-11-10 14:52:52 EST
The installclass translation files for product variants are provided in product.img files in the tree. The attached patches modify anaconda so that those translation files are used just for the product variant strings.
Created attachment 437281 [details]
The one thing that worries me about the second blob is that we'll be passing yum an argument with a space in it, and that argument could get used for making directories and various other things. Have you tested and verified that yum doesn't act too weirdly?
Good point. I chatted with Seth and a space in the repo ID will break yum but a space in the repo Name is fine. Unfortunately, at the moment getAddons() only returns a single value per addon which is used to set both the ID and Name. So, we either need to:
a) modify getAddons() to return separate ID and Name strings
b) modify doConfigSetup() to string spaces from the repo ID
c) revert that part of the patch altogether and translate the string without the space
I don't think c) will fly from a UI perspective. Either a) or b) is fine with me.
(a) seems the easiest and least error prone to me.
My vote is for (a) as well. Chris, reassigning to you since you did _getAddons.
Sent a patch to a-m-l:
Tested with lang=de, and all the tasks are translated. Repo names aren't, but the only repo I've got is "Red Hat Enterprise Linux" and that's a proper name. Packages are now installing.
This looks good in the 0810.n.0 nightly. Everything is translated including the repo names except for the "Red Hat Enterprise Linux" repo, which is expected.
Created attachment 439912 [details]
screenshot in French
Tested with 0818.0 build and lang=fr. All tasks and repo names are translated except Red Hat Enterprise Linux (base repo) which is expected. See attached screenshot.
Red Hat Enterprise Linux 6.0 is now available and should resolve
the problem described in this bug report. This report is therefore being closed
with a resolution of CURRENTRELEASE. You may reopen this bug report if the
solution does not work for you.