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 travis OSX build #29

Merged
merged 1 commit into from
Oct 10, 2017
Merged

Conversation

psychocrypt
Copy link
Collaborator

@psychocrypt psychocrypt commented Oct 8, 2017

fix broken travis OSX build

/usr/local/Homebrew/Library/Homebrew/brew.rb:12:in `<main>': Homebrew must be run under Ruby 2.3! (RuntimeError)
  • This PR is currently under testing, please do not merge

@psychocrypt psychocrypt added the bug label Oct 8, 2017
@psychocrypt
Copy link
Collaborator Author

psychocrypt commented Oct 8, 2017

The issue is coming from wrong characters within the environment variable PATH. (Homebrew/brew#2896)

ON travis PATH is:

Users/travis/.rvm/gems/ruby-2.4.1/bin:/Users/travis/.rvm/gems/ruby-2.4.1@global/bin:/Users/travis/.rvm/rubies/ruby-2.4.1/bin:/Users/travis/.rvm/bin:/Users/travis/bin:/Users/travis/.local/bin:/Users/travis/.nvm/versions/node/v6.9.1/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin

wrong char is @

UPDATE The @ within the pass is not the problem, something went wrong with brew update

@psychocrypt psychocrypt force-pushed the fix-travisOSXBuild branch 12 times, most recently from e0cb171 to 42c2325 Compare October 8, 2017 20:26
add `brew update` to avoid that `brew tap homebrew/science` crash
@psychocrypt psychocrypt changed the title [WIP] fix travis OSX build fix travis OSX build Oct 8, 2017
@psychocrypt
Copy link
Collaborator Author

ping: @fireice_uk could you please merge this PR, than I can rebase all other that the CI is passed without errors.

@fireice-uk
Copy link
Owner

You somehow managed to screw up the ping (it is in bold when it works) @psychocrypt

@fireice-uk fireice-uk merged commit 9af8d7f into fireice-uk:dev Oct 10, 2017
@psychocrypt
Copy link
Collaborator Author

psychocrypt commented Oct 10, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants