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 guides build #31039

Merged
merged 1 commit into from
Nov 3, 2017
Merged

Fix guides build #31039

merged 1 commit into from
Nov 3, 2017

Conversation

y-yagi
Copy link
Member

@y-yagi y-yagi commented Nov 3, 2017

Summary

Specify bundler version in template files.

We have already specified to install bundler 1.15.4 in .travis.yml.
https://github.com/rails/rails/blob/master/.travis.yml#L31..L32

However, bundler 1.16.0 may be used in the test.
https://travis-ci.org/rails/rails/jobs/296582467#L2208

The test failed due to this influence.
In order to avoid this, specifying bundler version in bug report templates.

@rails-bot
Copy link

r? @georgeclaghorn

(@rails-bot has picked a reviewer for you, use r? to override)

@y-yagi y-yagi force-pushed the fix_guides_build branch 7 times, most recently from 43f4edc to 5ba800d Compare November 3, 2017 11:37
@yahonda
Copy link
Member

yahonda commented Nov 3, 2017

I prefer this change since we can run CI using the latest version of Bundler 1.16.0 by isolating guides.
Also, it would help users who want to create a test case, they likely install bundler without specifying any version then install 1.16.0.

@kamipo
Copy link
Member

kamipo commented Nov 3, 2017

Is this change necessary for *_gem.rb templates? Actually *_gem.rb templates would not fail the bundle install without this change. Looks like the gem "rails", github: "rails/rails" line with bundler 1.16.0 will cause the failure.

We have already specified to install `bundler` 1.15.4 in `.travis.yml`.
https://github.com/rails/rails/blob/master/.travis.yml#L31..L32

However, `bundler` 1.16.0 may be used in the test.
https://travis-ci.org/rails/rails/jobs/296582467#L2208

The test failed due to this influence.
In order to avoid this, specifying `bundler` version in bug report
templates.
@y-yagi
Copy link
Member Author

y-yagi commented Nov 3, 2017

Looks like the gem "rails", github: "rails/rails" line with bundler 1.16.0 will cause the failure.

I think that it is correct. However, since *_gem.rb also failed, so I added it.
But I may have made a mistake. Remove from *_gem.rb and try running the test again.

@kamipo
Copy link
Member

kamipo commented Nov 3, 2017

Confirmed the running bug templates has passed.
https://travis-ci.org/rails/rails/jobs/296782070

@kamipo kamipo changed the title [WIP]Fix guides build Fix guides build Nov 3, 2017
@kamipo kamipo merged commit 015239a into rails:master Nov 3, 2017
kamipo added a commit that referenced this pull request Nov 3, 2017
This reverts commit 6f481e0, reversing
changes made to 592f790.

In favor of #31039.
@y-yagi y-yagi deleted the fix_guides_build branch November 3, 2017 21:42
yahonda added a commit to yahonda/oracle-enhanced that referenced this pull request Nov 10, 2017
Related with rails/rails#31039

$ ruby -v
ruby 2.4.2p198 (2017-09-14 revision 59899) [x86_64-linux]
$ bundler -v
Bundler version 1.16.0
$

```ruby
$ cd guides/bug_report_templates/
$ ruby active_record_gem.rb
... snip ...
/home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:28:in `block in setup': rake-12.2.1 is missing. Run `bundle install` to get it. (Bundler::GemNotFound)
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `map'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `setup'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/inline.rb:70:in `gemfile'
        from active_record_gem.rb:10:in `<main>'
$
```

```ruby
$ ruby active_record_gem_spec.rb
... snip ...
/home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:28:in `block in setup': rake-12.2.1 is missing. Run `bundle install` to get it. (Bundler::GemNotFound)
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `map'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `setup'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/inline.rb:70:in `gemfile'
        from active_record_gem_spec.rb:10:in `<main>'
$
```
yahonda added a commit to yahonda/oracle-enhanced that referenced this pull request Nov 10, 2017
Related to rails/rails#31039

$ ruby -v
ruby 2.4.2p198 (2017-09-14 revision 59899) [x86_64-linux]
$ bundler -v
Bundler version 1.16.0
$

```ruby
$ cd guides/bug_report_templates/
$ ruby active_record_gem.rb
... snip ...
/home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:28:in `block in setup': rake-12.2.1 is missing. Run `bundle install` to get it. (Bundler::GemNotFound)
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `map'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `setup'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/inline.rb:70:in `gemfile'
        from active_record_gem.rb:10:in `<main>'
$
```

```ruby
$ ruby active_record_gem_spec.rb
... snip ...
/home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:28:in `block in setup': rake-12.2.1 is missing. Run `bundle install` to get it. (Bundler::GemNotFound)
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/2.4.0/forwardable.rb:229:in `each'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `map'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/runtime.rb:26:in `setup'
        from /home/yahonda/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/inline.rb:70:in `gemfile'
        from active_record_gem_spec.rb:10:in `<main>'
$
```
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

5 participants