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

Nokogiri Major Version Dependency #90

Merged
merged 3 commits into from
Sep 18, 2014
Merged

Nokogiri Major Version Dependency #90

merged 3 commits into from
Sep 18, 2014

Conversation

highsineburgh
Copy link
Contributor

Changes to allow the installation of the plugin with the latest version of Vagrant which is bundled with Nokogiri 1.6.x. The dependency would allow all minor versions of Nokogiri after 1.5 to used.

The denpendency check for nokogiri was updated to allow all versions of nokogiri greater than 1.5.10 as the version bundled with vagrant is currently 1.6.x.
Switched from accepting all later versions of Nokogiri to all 1.x versions.
# force the use of Nokogiri 1.5.10 to prevent conflicts with older versions of zlib
s.add_dependency 'nokogiri', '~>1.5.10'
# force the use of Nokogiri 1.5.x to prevent conflicts with older versions of zlib
s.add_dependency 'nokogiri', '~>1.5'
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do you use ~>1.5 instead of optimistic dependency >=1.5.10?
That would allow reusing version 1.6 already loaded by Vagrant core.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Michael,

I switched from the >=1.5.10 to ~>1.5 to keep the dependency the same major
version of Nokogiri (i.e. 1.x). This seemed a safer approach than allowing
any version of Nokogiri greater than 1.5.10 which could potentially allow
version 2.x meet the dependency.

Cheers,
On Sep 16, 2014 11:54 AM, "Michael Kuzmin" [email protected]
wrote:

In vSphere.gemspec:

@@ -11,8 +11,8 @@ Gem::Specification.new do |s|
s.summary = 'VMWare vSphere provider'
s.description = 'Enables Vagrant to manage machines with VMWare vSphere.'

  • force the use of Nokogiri 1.5.10 to prevent conflicts with older versions of zlib

  • s.add_dependency 'nokogiri', '~>1.5.10'
  • force the use of Nokogiri 1.5.x to prevent conflicts with older versions of zlib

  • s.add_dependency 'nokogiri', '~>1.5'

Why do you use this instead of optimistic dependency >=1.5.10?
That would allow reusing version 1.6 already loaded by Vagrant core.


Reply to this email directly or view it on GitHub
https://github.com/nsidc/vagrant-vsphere/pull/90/files#r17618322.

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.

3 participants