Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix sanity check dep mvz #2241

Closed
wants to merge 13 commits into from
Closed

Fix sanity check dep mvz #2241

wants to merge 13 commits into from

Conversation

mvz
Copy link
Contributor

@mvz mvz commented Dec 18, 2019

Trying stuff to fix #2231 (comment)

benoittgt and others added 8 commits December 18, 2019 17:04
Avoid:
Failures:

  1) Verify required rspec dependencies passes when libraries are required
     Got 1 failure and 1 other error:

     1.1) Failure/Error: expect(`bundle exec #{script} 2>&1`).to be_empty
            expected `"WARNING:  open-ended dependency on activesupport (>= 4.2) is not recommended\n  if activesupport is ... 'railties', '~> 4.2'\nWARNING:  See http://guides.rubygems.org/specification-reference/ for help\n".empty?` to return true, got false
On Bundler 2.1.0 the loading of rubygems has been rewritten. It breaks
our existing usage of command execution. The --disable=gem flag is
problematic because we need to load rubygems in Bundler. It is
not explicitly loaded in Bundle 2.1.0

Related:
  - rubygems/bundler#7487
@JonRowe
Copy link
Member

JonRowe commented Dec 20, 2019

Closing in favour of #2244 thanks for your work here @mvz, it helped!

@JonRowe JonRowe closed this Dec 20, 2019
@mvz mvz deleted the fix-sanity-check-dep-mvz branch December 20, 2019 13:47
@mvz mvz restored the fix-sanity-check-dep-mvz branch January 1, 2020 13:49
@mvz mvz deleted the fix-sanity-check-dep-mvz branch January 1, 2020 13:56
@benoittgt benoittgt mentioned this pull request Jan 3, 2020
benoittgt added a commit that referenced this pull request Jan 3, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
benoittgt added a commit that referenced this pull request Jan 4, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
JonRowe pushed a commit that referenced this pull request Jan 12, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
JonRowe pushed a commit that referenced this pull request Jan 12, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
pirj pushed a commit that referenced this pull request Jun 29, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
pirj pushed a commit that referenced this pull request Jul 20, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
pirj pushed a commit that referenced this pull request Jul 21, 2020
With new bundler update like 2.1.2 and 2.1.3 we saw issues on the CI
after running `bundle exec cucumber`:

```
+/home/travis/.rvm/gems/ruby-2.6.3/gems/bundler-2.1.3/lib/bundler/runtime.rb:312:in `check_for_activated_spec!': You have already activated bundler 2.1.2, but your Gemfile requires bundler 2.1.3. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)
```

Using the patch from mvz #2241
reduce the amount of error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants