Bug #4916

Hammer 0.1.0 is not executable on debian systems

Added by Tomáš Strachota over 3 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Tomáš Strachota
Category:Hammer core
Target version:Foreman - Sprint 22
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-

Description

There's a problem with finding locales which prevents hammer from starting.

# hammer
/usr/lib/ruby/vendor_ruby/fast_gettext/translation_repository/base.rb:36:in `find_files_in_locale_folders': path /usr/lib/ruby/vendor_ruby/hammer_cli/../../locale cound not be found! (RuntimeError)
    from /usr/lib/ruby/vendor_ruby/fast_gettext/translation_repository/mo.rb:25:in `find_and_store_files'
    from /usr/lib/ruby/vendor_ruby/fast_gettext/translation_repository/mo.rb:9:in `initialize'
    from /usr/lib/ruby/vendor_ruby/fast_gettext/translation_repository.rb:14:in `new'
    from /usr/lib/ruby/vendor_ruby/fast_gettext/translation_repository.rb:14:in `build'
    from /usr/lib/ruby/vendor_ruby/fast_gettext.rb:28:in `add_text_domain'
    from /usr/lib/ruby/vendor_ruby/hammer_cli/i18n.rb:81:in `add_domain'
    from /usr/lib/ruby/vendor_ruby/hammer_cli/i18n.rb:94:in `<top (required)>'
    from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from /usr/bin/hammer:9:in `<main>'

Default gem installation on Debian puts all gems into one directory. Hammer tries to find locales in a directory that does not exist.
We should update hammer's i18n code to accept more translation locations.

Associated revisions

Revision 77c62f8e
Added by Tomas Strachota over 3 years ago

Fixes #4916 - skip missing translation domains

Revision 4341058b
Added by Tomas Strachota over 3 years ago

Merge pull request #103 from tstrachota/missing_domains

Fixes #4916 - skip missing translation domains

History

#1 Updated by Tomáš Strachota over 3 years ago

  • Status changed from Assigned to Ready For Testing

#2 Updated by Anonymous over 3 years ago

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

Applied in changeset foreman:hammer-cli|commit:77c62f8e9410c7ab743f0b4cbc7ff9c2a147c98a.

Also available in: Atom PDF