Project

General

Profile

Bug #16101

Following the README in setting up hammer-cli-katello, you get a gem dependency conflict

Added by David Davis about 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hammer
Target version:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

If you follow hammer-cli-katello's README, you get a bundler error:

[!] There was an error parsing `Gemfile`: You cannot specify the same gem twice coming from different sources.
You specified that hammer_cli (>= 0) should come from https://github.com/theforeman/hammer-cli.git (at master) and source at `../hammer-cli`
. Bundler cannot continue.

 #  from /home/vagrant/Projects/hammer-cli-katello/Gemfile:25
 #  -------------------------------------------
 #    local_gemfile = File.join(File.dirname(__FILE__), file_name)
 >    self.instance_eval(Bundler.read_file(local_gemfile)) if File.exist?(local_gemfile)
 #  end
 #  -------------------------------------------

It's because we added these lines to Gemfile:

group :development, :test do
  gem 'hammer_cli_foreman', :git => 'https://github.com/theforeman/hammer-cli-foreman.git'
  gem 'hammer_cli', :git => 'https://github.com/theforeman/hammer-cli.git'
end

I propose we remove these lines and document how users can use either local checkouts or git repos.

Associated revisions

Revision f46fbd64 (diff)
Added by David Davis about 3 years ago

Fixes #16101 - Update Gemfile and README setup info (#442)

Revision 8f19345a (diff)
Added by David Davis almost 3 years ago

Refs #16101 - Remove gem git requirements (#443)

History

#1 Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/442 added

#2 Updated by Anonymous about 3 years ago

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

#3 Updated by Justin Sherrill about 3 years ago

  • Legacy Backlogs Release (now unused) set to 162

#4 Updated by The Foreman Bot almost 3 years ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/443 added

Also available in: Atom PDF