Re: [Change Request] Fix FAS Chinese translation

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

 



On Friday, May 13, 2011 08:16:30 PM Toshio Kuratomi wrote:
> On Fri, May 13, 2011 at 09:07:18PM -0400, Ricky Zhou wrote:
> > Currently, selecting the Chinese translation of FAS causes FAS to
> > traceback, due to an bug in Python's gettext implementation
> > (http://bugs.python.org/issue1448060).  I'd like to hotfix this by
> > replacing the Chinese MO file with a newer version.  This is pretty low
> > risk, since the Chinese translation can't become any more broken than it
> > is now :-)
> > 
> > Upstream commit:
> > http://git.fedorahosted.org/git/?p=fas.git;a=commit;h=8a55ecb7d887014dabd
> > dc3ce23be53cff31b7c9c
> > 
> > Puppet patch:
> > diff --git a/modules/hotfix/files/fas/zh_CN.mo
> > b/modules/hotfix/files/fas/zh_CN.mo new file mode 100644
> > index 0000000..dba956f
> > Binary files /dev/null and b/modules/hotfix/files/fas/zh_CN.mo differ
> > diff --git a/modules/hotfix/manifests/init.pp
> > b/modules/hotfix/manifests/init.pp index 5e83163..179d687 100644
> > --- a/modules/hotfix/manifests/init.pp
> > +++ b/modules/hotfix/manifests/init.pp
> > @@ -14,6 +14,13 @@ class hotfix::python-fedora-fas2 {
> > 
> >  }
> >  
> >  class hotfix::fas {
> > 
> > +    file { "/usr/share/locale/zh_CN/LC_MESSAGES/fas.mo":
> > +        owner    => "root",
> > +        group    => "root",
> > +        mode     => 0644,
> > +        source   => "puppet:///hotfix/fas/zh_CN.mo",
> > +        requires => Package["fas"],
> > +    }
> > 
> >  }
> >  
> >  class hotfix::pkgdb {
> 
> +1
> 
> Thanks for fixing this,
> 
> -Toshio
+1

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

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux