Skip to content

Build updates #156

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

Merged
merged 3 commits into from
May 16, 2016
Merged

Build updates #156

merged 3 commits into from
May 16, 2016

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented May 9, 2016

So it seems Bundler 1.12 is still broken for us, I expect us to be able to remove the version select at some point, but currently we can't install json on 2.3.1 on travis, and it also seems to break appveyor with warnings.

@JonRowe
Copy link
Member Author

JonRowe commented May 10, 2016

@@ -6,7 +6,8 @@ cache:
before_install:
- "script/clone_all_rspec_repos"
# Note this doesn't work on JRUBY 2.0.0 mode so we don't do it, the excluded versions are broken on Ruby 2.3
- if [ "jruby" != "$TRAVIS_RUBY_VERSION" ]; then gem install bundler --version "!= 1.12.0, != 1.12.1"; fi
- if [ "jruby" != "$TRAVIS_RUBY_VERSION" ]; then gem install bundler --version "!= 1.12.0, != 1.12.1, != 1.12.2, != 1.12.3"; fi
install: if [ "jruby" != "$TRAVIS_RUBY_VERSION" ]; then bundle _1.11.2_ install --binstubs --standalone --without documentation --path ../bundle; else bundle install --binstubs --standalone --without documentation --path ../bundle; fi
Copy link
Member

Choose a reason for hiding this comment

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

Honestly, it seems simpler to just install 1.11 (and specify just that version) util there's a new release that works.

Copy link
Member

Choose a reason for hiding this comment

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

Also, instead of putting the whole bundle command, you can just alias bundle to bundle _1.11.2_. See 35ac13f for an example.

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated, 1.12.4 will fix this anyway :)

@JonRowe JonRowe merged commit 2b61054 into master May 16, 2016
@JonRowe JonRowe deleted the build_updates branch May 16, 2016 11:18
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.

2 participants