foreman icon indicating copy to clipboard operation
foreman copied to clipboard

Fixes #36574 - Update fast_gettext to ~> 2.1

Open ekohl opened this issue 2 years ago • 5 comments

ekohl avatar Jul 07 '23 14:07 ekohl

I needed to avoid translating during startup because it led to infinite recursion. https://github.com/theforeman/foreman/pull/9771 is that PR and included here.

ekohl avatar Jul 07 '23 15:07 ekohl

There's another case here: https://github.com/theforeman/foreman/blob/df3a0b0d970b229887e2d371568d16c8143c5aae/app/models/operatingsystem.rb#L82

The human method is here: https://github.com/grosser/gettext_i18n_rails/blob/master/lib/gettext_i18n_rails/active_model/name.rb

I'll dig a bit deeper.

ekohl avatar Jul 07 '23 15:07 ekohl

Glad to see this getting tackled, for both Ruby / Rails reasons and to fix a conflict we've had in packaging for a while now:

Found conflict for rubygem-fast_gettext: 1.8.0 (foreman) != 2.3.0 (hammer-cli)

ehelms avatar Aug 01 '23 18:08 ehelms

Thanks. Given the trace of the infinite loop I saw it makes sense. I'll see about giving it a spin. If that works, then I can also drop the 36757 commits from this PR. They'll still be good, but no longer a blocker to this PR.

ekohl avatar Oct 16 '23 14:10 ekohl

In theory this is now ready, but I'd like to postpone this until after branching.

ekohl avatar Nov 16 '23 17:11 ekohl

We've branched twice. Let's get this in.

ekohl avatar Feb 29 '24 10:02 ekohl

@ekohl, could you please fix rubocop failures? So we can see if the tests are passing at least.

ofedoren avatar Feb 29 '24 12:02 ofedoren

Updated, but didn't have time to test this myself. I'll check back what CI says after lunch.

ekohl avatar Mar 01 '24 11:03 ekohl

Everything seems to be green, except that it cannot be built for some reason :/

ofedoren avatar Mar 01 '24 12:03 ofedoren

Yeah, RPM will fail as it has no 2.1 package yet.

/usr/share/rubygems/rubygems/dependency.rb:313:in `to_specs': Could not find 'fast_gettext' (~> 2.1) - did find: [fast_gettext-1.8.0] (Gem::MissingSpecVersionError)

evgeni avatar Mar 01 '24 12:03 evgeni

Let's see: https://github.com/theforeman/foreman-packaging/pull/10515

evgeni avatar Mar 01 '24 12:03 evgeni

@ofedoren can I get an ACK on the code? I have not the slightest idea about gettext :/

evgeni avatar Mar 04 '24 06:03 evgeni