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

update rubygems per #1647 to avoid bundle exec failure on jruby in 1.9 mode #1658

Merged
merged 3 commits into from
May 15, 2012
Merged

update rubygems per #1647 to avoid bundle exec failure on jruby in 1.9 mode #1658

merged 3 commits into from
May 15, 2012

Conversation

wenzowski
Copy link

No description provided.

@parndt
Copy link
Member

parndt commented May 12, 2012

Doesn't this make rbx fail?

@wenzowski
Copy link
Author

It does appear to make rbx fail on something different (bundling bcrypt), but wasn't it failing before?

@travisbot
Copy link

This pull request passes (merged dbbc9139 into b6d85c7).

@parndt
Copy link
Member

parndt commented May 15, 2012

Yes failing but still actually running http://travis-ci.org/#!/resolve/refinerycms/jobs/1330225

@parndt
Copy link
Member

parndt commented May 15, 2012

Okay @mpapis has come up with a beautiful solution to our problem in 96bfe084194edfc6c7d4ccce53d15a7bf562d0d5 so I'll merge yours + another commit for that later on once the build completely finishes.

@wenzowski
Copy link
Author

that would indeed be the smart way to do it

@wenzowski
Copy link
Author

merged @mpapis patch and rebased from master

@travisbot
Copy link

This pull request fails (merged 19e434c into df163ac).

@travisbot
Copy link

This pull request passes (merged 8721fc2 into df163ac).

parndt added a commit that referenced this pull request May 15, 2012
update rubygems per #1647 to avoid `bundle exec` failure on jruby in 1.9 mode
@parndt parndt merged commit 1b0daab into refinery:master May 15, 2012
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