Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

Prevent bad version of nokogiri and fix build #2140

Merged
merged 1 commit into from
Dec 24, 2015
Merged

Prevent bad version of nokogiri and fix build #2140

merged 1 commit into from
Dec 24, 2015

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Dec 24, 2015

No description provided.

JonRowe added a commit that referenced this pull request Dec 24, 2015
Prevent bad version of nokogiri and fix build
@JonRowe JonRowe merged commit d148026 into master Dec 24, 2015
@JonRowe
Copy link
Member Author

JonRowe commented Dec 24, 2015

Merged despite error because it needs all three simultaneously. See rspec/rspec-rails#1510 rspec/rspec-support#258 and #2140

@JonRowe JonRowe deleted the pin_nokogiri branch December 24, 2015 06:05
@@ -42,7 +42,7 @@ Gem::Specification.new do |s|
s.add_development_dependency "minitest", "~> 5.3"
s.add_development_dependency "aruba", "~> 0.6.2" # 0.7 is broken on ruby 1.8.7

s.add_development_dependency "nokogiri", (RUBY_VERSION < '1.9.3' ? "1.5.2" : ["~> 1.5", "!= 1.6.6.3", "!= 1.6.6.4"])
s.add_development_dependency "nokogiri", (RUBY_VERSION < '1.9.3' ? "1.5.2" : ["~> 1.5", "!= 1.6.6.3", "!= 1.6.6.4", "!= 1.6.7.1"])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow. Crazy the hoops we have to jump through here :(.

Makes me wonder if there's something different we can use...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's because we disable ruby gems apparently

MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
Prevent bad version of nokogiri and fix build
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants