On Sun, 2008-03-16 at 05:23 +0530, Kulbir Saini wrote:
> Hi all!
> 
>       We were not checking for possible errors while converting to utf-8. I 
> have attached a patch for the same. The patch was generated using 'git 
> diff'.
>       This is my first patch. Hope I generated it the right way :)

 Yeh, however:

1. At the very least it should only be catching "UnicodeDecodeError".

2. That unicode() call was put there because otherwise
pkg.summary/pkg.description for libselinux caused problems in de_DE
(among other locale problems -- as ® wasn't valid for non-unicode).

 ® also looks like it's the problem for you. So what locale do you have
(please post output of the locale command)?

 Maybe we can also work around it by only calling unicode() if we get
"translated data" from gettext.dgettext() ... as I assume whatever
locale you have doesn't have a translation.
 I'm loath to have so many edge cases everywhere we want to output
locale translated pkg.summary/description.

-- 
James Antill <[EMAIL PROTECTED]>
Red Hat

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Yum-devel mailing list
[email protected]
https://lists.dulug.duke.edu/mailman/listinfo/yum-devel

Reply via email to