Project

General

Custom queries

Profile

Actions

Refactor #8463

closed

Remove gettext_i18n_rails_js from runtime dependencies

Added by Dominic Cleal over 10 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Packaging
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

gettext_i18n_rails_js is currently listed in bundler.d/i18n.rb (and assets.rb) which makes it a runtime dependency, but I don't think it needs to be. It only provides two things: rake tasks (and parsers), and assets which are precompiled.

gettext_i18n_rails_js should be in the assets + development groups only, so it's included when we precompile assets and again for developers who need those rake tasks to build locale files. Ensure that lib/tasks/gettext.rake is updated so it handles the gem being missing when it requires gettext_i18n_rails_js/tasks.

I'd probably suggest rearranging the i18n group a bit and moving fast_gettext, gettext_i18n_rails and i18n_data (see also #8462) to Gemfile as they're required deps, it's not an optional feature - then i18n.rb can be deleted.


Related issues 1 (0 open1 closed)

Related to Foreman - Bug #8462: Remove or use i18n_data dependencyClosedShlomi Zadok11/20/2014Actions
#1

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #8462: Remove or use i18n_data dependency added
#3

Updated by Dominic Cleal over 10 years ago

  • Status changed from New to Assigned
  • Assignee set to Dominic Cleal
#4

Updated by The Foreman Bot over 10 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1981 added
  • Pull request deleted ()
#5

Updated by Dominic Cleal over 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
#6

Updated by Dominic Cleal over 10 years ago

  • Translation missing: en.field_release set to 28
Actions

Also available in: Atom PDF