[Pkg-puppet-devel] Bug#892737: Bug#892737: Bug#892737: puppet: /usr/lib/ruby/vendor_ruby/puppet/gettext/config.rb:156:in `copy_default_translations': undefined method `chain'

[Apollon Oikonomopoulos]

Control: tags -1 pending

Hi Georg,

On 02:30 Tue 13 Mar     , Georg Faerber wrote:
> Hi Apollon,
> 
> On 18-03-13 00:19:06, Apollon Oikonomopoulos wrote:
> > On 23:09 Mon 12 Mar     , Georg Faerber wrote:
> > > On 18-03-12 23:28:20, Apollon Oikonomopoulos wrote:
> > > > So, the version of ruby-gettext-setup is pretty outdated and 
> > > > predates
> > > > Puppet's i18n system. When pulled in via i18n-enabled Puppet modules
> > > > (e.g. ruby-puppet-forge), it will completely hijack Puppet's gettext
> > > > default domain, causing breakage. This has been fixed upstream[1] in
> > > > 0.17.
> > > > 
> > > > We should upgrade ruby-gettext-setup to the latest upstream version.
> > > 
> > > I could take care of this, tomorrow. Just need someone to sponsor the
> > > upload.
> > 
> > Please do! I'd be glad to sponsor this, ping me when you're ready or
> > if you need any help.
> 
> I've pushed in git to branch debian/0.30-1. Please especially review
> eeff22b6, in particular the changes I've made to
> spec/lib/tasks/gettext_rake_spec.rb. 
> 
> (This file loads an rake task contained in lib/. I'm not really sure if
> I'm happy with my proposed change, however, I've tried various methods
> and didn't found anything else working besides this. Reasoning for this
> change: Upstreams code doesn't work if specs are run via autopkgtest, as
> lib/ is moved away during autopkgtest runs. In case you would like to
> change this: Please go ahead.)

Looks good to me! I've pushed an additional change (unexport LANGUAGE in 
d/rules) to fix a test failure occurring on my system. I'll upload the 
package shortly.

Thanks for your work!
Apollon