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 2.0 installer script #100

Conversation

Larusso
Copy link
Contributor

@Larusso Larusso commented Nov 28, 2019

Hey I made some adjustments to your changes for the new installer
package.

  • Add missing fastlane_shim file
  • remove ssl certificate environment line from bundle-env
  • added option switch -u to skip prompts for autoupdate/install
  • made the fastlane_shim symlink-able

Hey I made some adjustments to your changes for the new installer
package.

* Add missing `fastlane_shim` file
* remove ssl certificate environment line from `bundle-env`
* added option switch `-u` to skip prompts for autoupdate/install
@googlebot

This comment has been minimized.

@Larusso

This comment has been minimized.

@googlebot

This comment has been minimized.

@snatchev
Copy link
Member

snatchev commented Dec 3, 2019

@Larusso Thanks for the PR. Just reviewed the changes and I don't see changes with regards to ssl. Was that dropped?

Pinging @joshdholtz

@Larusso
Copy link
Contributor Author

Larusso commented Dec 4, 2019

@snatchev I removed this line from bundle-env

export SSL_CERT_FILE=$PREFIX/share/cacert.pem

In this PR I still kep the brew open-ssl download logic. Because I don't know the exact reason why this is supposed to be better than letting ruby-build installing open-ssl.

Copy link
Member

@joshdholtz joshdholtz left a comment

Choose a reason for hiding this comment

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

Thanks for this fix! Good catch on removing the SSL_CERT_FILE. That was a left over from my change 🙃 And thanks for putting back in the fastlane_shim ❤️

@joshdholtz joshdholtz merged commit 5772885 into fastlane:2.0-build-and-install-ruby-on-brew-install Dec 4, 2019
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

4 participants