Re: [PATCH] 470733 - yet more unicode errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 11, 2008 at 03:53:01PM -0500, Chris Lumens wrote:
> > from yum.misc import to_unicode, to_utf8, to_unicode_maybe, to_str
> >
> > in theory, those should help things a good bit.
> 
> Even better, because that's less code we have to maintain.

I like it more now.

> 
> - Chris
> 
> 
> diff --git a/yuminstall.py b/yuminstall.py
> index a962639..694bffe 100644
> --- a/yuminstall.py
> +++ b/yuminstall.py
> @@ -39,6 +39,7 @@ import yum
>  import iniparse
>  from yum.constants import *
>  from yum.Errors import *
> +from yum.misc import to_unicode
>  from yum.yumRepo import YumRepository
>  from backend import AnacondaBackend
>  from product import *
> @@ -71,17 +72,21 @@ def size_string (size):
>      def number_format(s):
>          return locale.format("%s", s, 1)
>  
> +    retval = None
> +
>      if size > 1024 * 1024:
>          size = size / (1024*1024)
> -        return _("%s MB") %(number_format(size),)
> +        retval = _("%s MB") %(number_format(size),)
>      elif size > 1024:
>          size = size / 1024
> -        return _("%s KB") %(number_format(size),)        
> +        retval = _("%s KB") %(number_format(size),)
>      else:
>          if size == 1:
> -            return _("%s Byte") %(number_format(size),)                    
> +            retval = _("%s Byte") %(number_format(size),)
>          else:
> -            return _("%s Bytes") %(number_format(size),)
> +            retval = _("%s Bytes") %(number_format(size),)
> +
> +    return to_unicode(retval)
>  
>  class AnacondaCallback:
>  
> @@ -155,14 +160,12 @@ class AnacondaCallback:
>              repo = self.repos.getRepo(po.repoid)
>  
>              pkgStr = "%s-%s-%s.%s" % (po.name, po.version, po.release, po.arch)
> -            s = _("<b>Installing %s</b> (%s)\n") %(pkgStr, size_string(hdr['size']))
> -            summary = gettext.ldgettext("redhat-dist", hdr['summary'] or "")
> -            if type(summary) != unicode:
> -                summary = unicode(summary, encoding='utf-8')
> +            s = to_unicode(_("<b>Installing %s</b> (%s)\n")) %(pkgStr, size_string(hdr['size']))
> +            summary = to_unicode(gettext.ldgettext("redhat-dist", hdr['summary'] or ""))
>              s += summary.strip()
>              self.progress.set_label(s)
>  
> -            self.instLog.write(self.modeText % pkgStr)
> +            self.instLog.write(self.modeText % str(pkgStr))
>  
>              self.instLog.flush()
>              self.openfile = None
> @@ -645,9 +648,7 @@ class AnacondaYum(YumSorter):
>          else:
>              buttons = [_("Re_boot"), _("_Retry")]
>  
> -        pkgFile = os.path.basename(package.returnSimple('relativepath'))
> -        if type(pkgFile) != unicode:
> -            pkgFile = unicode(pkgFile, encoding='utf-8')
> +        pkgFile = to_unicode(os.path.basename(package.returnSimple('relativepath')))
>  
>          rc = self.anaconda.intf.messageWindow(_("Error"),
>                     _("The file %s cannot be opened.  This is due to a missing "
> @@ -839,11 +840,8 @@ class AnacondaYum(YumSorter):
>              msg = _("There was an error running your transaction for "
>                      "the following reason(s): %s.\n") % ', '.join(uniqueProbs.values())
>  
> -            if type(spaceprob) != unicode:
> -                spaceprob = unicode(spaceprob, encoding='utf-8')
> -
> -            if type(fileprob) != unicode:
> -                fileprob = unicode(fileprob, encoding='utf-8')
> +            spaceprob = to_unicode(spaceprob)
> +            fileprob = to_unicode(fileprob)
>  
>              if len(self.anaconda.backend.getRequiredMedia()) > 1:
>                  intf.detailedMessageWindow(_("Error Running Transaction"),
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
David Cantrell <dcantrell@xxxxxxxxxx>
Red Hat / Honolulu, HI

Attachment: pgpr29oye6xZf.pgp
Description: PGP signature

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux