Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Bump travis rubies #7036

Merged
6 commits merged into from
Mar 27, 2019
Merged

Bump travis rubies #7036

6 commits merged into from
Mar 27, 2019

Conversation

deivid-rodriguez
Copy link
Member

What was the end-user problem that led to this PR?

The problem was that hacks create confusion, even if they include TODO notes.

What was your diagnosis of the problem?

My diagnosis was that we can should upgrade to the latest ruby releases to reduce the number of hacks we need to maintain.

What is your fix for the problem, implemented in this PR?

My fix is to upgrade rubies and remove hacks.

Why did you choose this fix out of the possible options?

I chose this fix because it's a good change.

@colby-swandale
Copy link
Member

Restarted the build. Seems that the tgz for Ruby 2.6.2 wasn't up at the time of opening the PR.

@deivid-rodriguez
Copy link
Member Author

I still need to restore some of the hacks. The gem install --defaul etc:1.0.1 one is still needed :(

@deivid-rodriguez deivid-rodriguez force-pushed the bump_travis_rubies branch 3 times, most recently from 39c9a16 to d3095df Compare March 15, 2019 14:40
To avoid activation of the `etc` gem.
@deivid-rodriguez
Copy link
Member Author

Hei!

I managed to get this green and removed all hacks. After closer examination, it might be gem install etc:1.0.1 --default (in general, gem install <gem_with_extension> --default), and not the ruby-core installer that was wrong. So what the hack was doing was "breaking" the installed specification, and thus preventing the default gem from being activated.

This sounds like a better fix. It needs a patch to fileutils that I also proposed upstream.

@deivid-rodriguez
Copy link
Member Author

We should wait for the upstream PR, @hsbt could you have a look?

@deivid-rodriguez deivid-rodriguez requested a review from hsbt March 26, 2019 16:20
@hsbt
Copy link
Member

hsbt commented Mar 27, 2019

@bundlerbot r+

ghost pushed a commit that referenced this pull request Mar 27, 2019
7036: Bump travis rubies r=hsbt a=deivid-rodriguez

### What was the end-user problem that led to this PR?

The problem was that hacks create confusion, even if they include TODO notes.

### What was your diagnosis of the problem?

My diagnosis was that we can should upgrade to the latest ruby releases to reduce the number of hacks we need to maintain.

### What is your fix for the problem, implemented in this PR?

My fix is to upgrade rubies and remove hacks.

### Why did you choose this fix out of the possible options?

I chose this fix because it's a good change.

Co-authored-by: David Rodríguez <[email protected]>
@ghost
Copy link

ghost commented Mar 27, 2019

Build succeeded

@ghost ghost merged commit c7ac5b3 into master Mar 27, 2019
@ghost ghost deleted the bump_travis_rubies branch March 27, 2019 07:52
@hsbt
Copy link
Member

hsbt commented Mar 27, 2019

@deivid-rodriguez
Copy link
Member Author

Great! I opened #7068 to vendor that!

colby-swandale pushed a commit that referenced this pull request Apr 5, 2019
7036: Bump travis rubies r=hsbt a=deivid-rodriguez

The problem was that hacks create confusion, even if they include TODO notes.

My diagnosis was that we can should upgrade to the latest ruby releases to reduce the number of hacks we need to maintain.

My fix is to upgrade rubies and remove hacks.

I chose this fix because it's a good change.

Co-authored-by: David Rodríguez <[email protected]>
(cherry picked from commit d368aa9)
colby-swandale added a commit that referenced this pull request Jun 13, 2019
* 2-0-stable: (89 commits)
  fix changelog 2.0.2 typos
  add v2.0.2 changelog
  bump version to 2.0.2
  Merge #7199
  fix bug where bundler v3 is running a test for bundflet 2
  Merge #6798
  add bors configuation
  port GemHelper from master
  Merge #7080
  Merge #7089
  Merge #7068
  Merge #7036
  Merge #7067
  change Bundler 3 specs in travis to use RubyGems 3.0.3
  bump RubyGems v3 to the latest version on Travis
  Merge #6963
  Merge #7078
  Merge pull request #7061 from bundler/fix_circular_requires
  Merge #6864
  remove linting step in travis (it will still run in each build)
  ...
This pull request was closed.
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.

3 participants