Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

Modifying rhc to focus support on ruby1.8.7 #750

Merged
merged 1 commit into from Apr 22, 2016

Conversation

thrasher-redhat
Copy link
Contributor

With Ruby 1.8.7 and 1.9.3 being deprecated, many gems are dropping support for
them. These dependencies make it impossible for rhc to support both Ruby 1.8.7
and newer Ruby 2.x versions at the same time. With the upcoming move towards
OpenShift v3 - we will be focusing support on Ruby 1.8.7. We're going to try to
support as far forward as possible without breaking rhc on Ruby 1.8.7.

@thrasher-redhat
Copy link
Contributor Author

[test], please!

Possible things todo:

  • Determine how far forward we currently can support
  • Update README to reflect supported versions
  • Update travis testing to stop testing non-supported versions
  • Examine locking gem version in the Gemfile for non-gem installations...?

@thrasher-redhat
Copy link
Contributor Author

This PR will close several previous PRs attempting to solve the problem of supporting multiple versions.

#736 #739 #748 #749

@thrasher-redhat thrasher-redhat force-pushed the dependencyfix branch 3 times, most recently from c5bef91 to eb3bc43 Compare March 30, 2016 22:24
@thrasher-redhat
Copy link
Contributor Author

So since jenkins is testing with ruby 1.9.3, the addressable change is causing test failures. Need to see if these tests would fail with 1.8.7 (and addressable 2.3.2 or 2.3.8). Also may need to check the tests themselves to see if they are failures of functionality or if the test should be updated.

@thrasher-redhat thrasher-redhat force-pushed the dependencyfix branch 4 times, most recently from 0e880aa to d1a0c8a Compare April 21, 2016 19:18
@thrasher-redhat thrasher-redhat changed the title [WIP] Modifying rhc to focus support on ruby1.8.7 Modifying rhc to focus support on ruby1.8.7 Apr 21, 2016
@@ -30,7 +30,6 @@ Gem::Specification.new do |s|
sep
].join("\n")

s.add_dependency 'net-ssh', '>= 2.0.11', '<= 2.9.2'
s.add_dependency 'net-scp', '>= 1.1.2'
s.add_dependency 'net-ssh-multi','>= 1.2.0'
s.add_dependency 'archive-tar-minitar'
Copy link
Member

Choose a reason for hiding this comment

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

Didn't we want to lock down versions for all dependencies to the latest known working version? Or do we not mind handling locking them down once a version is released that breaks us?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm pretty sure in our meeting Dan said that locking EVERY version of EVERY dependency to a known good one would take a lot more effort than its worth.

Copy link
Member

Choose a reason for hiding this comment

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

Okay, I'm fine with this then. We just need to be aware that it is possible for the unlocked gems to cause issues in the future with updates.

With Ruby 1.8.7 and 1.9.3 being deprecated, many gems are dropping support for
these older versions of Ruby.  These dependencies make it impossible for rhc to
support leveraging rhc on both Ruby 1.8.7 and newer Ruby 2.x versions at the
same time.  With the upcoming move towards OpenShift v3 - we will be focusing
support on rhc built with Ruby 1.8.7.  This version of rhc may work with newer
versions of ruby, but the primary support will be 1.8.7.

Conditional building options have been added.  These will use the correct
dependencies at build time for the ruby version being used.  The standard rhc
gem from rubygems.org is built with Ruby 1.8.7, which can run into dependency
issues when running rhc with later versions of Ruby.  Users can build rhc from
source on the system in question to use the correct dependencies for their Ruby
version.
@thrasher-redhat
Copy link
Contributor Author

Now with 100% more backticks!

@openshift-bot
Copy link

Evaluated for online test up to 3fb8f05

@Miciah
Copy link
Contributor

Miciah commented Apr 22, 2016

Lovely jubbly! openshift-bot, please [merge]!

@openshift-bot
Copy link

Online Merge Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/merge_pull_requests/6727/) (Image: devenv_5786)

@openshift-bot
Copy link

Evaluated for online merge up to 3fb8f05

@openshift-bot
Copy link

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.

None yet

4 participants