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

add Homebrew instructions #895

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
5 participants
@btbytes

btbytes commented Jun 18, 2018

Purpose

Improve documentation

Related PRs

Only the submitted PR

Special notes

  • only documentation change. No tests were created/modified.
@TylerJewell

This comment has been minimized.

Show comment
Hide comment
@TylerJewell

TylerJewell Jun 18, 2018

Contributor

Hmm, I think this may not be the best place for this type of thing. I think it's because we should create a special section on the installation / download page for this information, and then avoid repeating this info on the Getting Started page. We are likely to have many more of these installation packages, and so it would get to be too noisy on the Getting Started.

Contributor

TylerJewell commented Jun 18, 2018

Hmm, I think this may not be the best place for this type of thing. I think it's because we should create a special section on the installation / download page for this information, and then avoid repeating this info on the Getting Started page. We are likely to have many more of these installation packages, and so it would get to be too noisy on the Getting Started.

@btbytes

This comment has been minimized.

Show comment
Hide comment
@btbytes

btbytes Jun 18, 2018

Considering ballerina is a developer tool and homebrew is the most popular way for developers to install software on a mac (AFAIK), brew install ballerina should perhaps take precedence over binary download + install.

btbytes commented Jun 18, 2018

Considering ballerina is a developer tool and homebrew is the most popular way for developers to install software on a mac (AFAIK), brew install ballerina should perhaps take precedence over binary download + install.

@TylerJewell

This comment has been minimized.

Show comment
Hide comment
@TylerJewell

TylerJewell Jun 18, 2018

Contributor

We had numerous back-and-forth about that very topic when we did our GTM planning offsite at the end of March.

We opted against making the package managers the default installation because we cannot guarantee (with many of them) that they get propagation to the proper version at the time of a release. So we are not able to put them through all of the verification processes that ensures quality and many package managers have different propagation of versions over time, along with some approval processes.

So the only guaranteed consistent way for all of the OS is the approach that we have - and then we wanted to iteratively evolve the downloads page with all of the package manager options available. It's a bit similar to how Rust has handled it, which is pretty elegant.

Contributor

TylerJewell commented Jun 18, 2018

We had numerous back-and-forth about that very topic when we did our GTM planning offsite at the end of March.

We opted against making the package managers the default installation because we cannot guarantee (with many of them) that they get propagation to the proper version at the time of a release. So we are not able to put them through all of the verification processes that ensures quality and many package managers have different propagation of versions over time, along with some approval processes.

So the only guaranteed consistent way for all of the OS is the approach that we have - and then we wanted to iteratively evolve the downloads page with all of the package manager options available. It's a bit similar to how Rust has handled it, which is pretty elegant.

@TylerJewell

This comment has been minimized.

Show comment
Hide comment
@TylerJewell

TylerJewell Jun 18, 2018

Contributor

BTW - thanks for getting involved and making the recommendation. This PR is just triggering the resurgence of all the discussions that were had around package managers, and need to figure out whether now is a good time to ask R&D core team to revisit how they add all of the package managers as part of our CI system, so that they magically show up everywhere they need to be.

Contributor

TylerJewell commented Jun 18, 2018

BTW - thanks for getting involved and making the recommendation. This PR is just triggering the resurgence of all the discussions that were had around package managers, and need to figure out whether now is a good time to ask R&D core team to revisit how they add all of the package managers as part of our CI system, so that they magically show up everywhere they need to be.

Show outdated Hide outdated website/two-column-pages/docs/learn/getting-started.md
@@ -30,6 +30,12 @@ If you are upgrading from an older version of Ballerina, you can simply download
The package automatically sets your PATH environment variable for you. You may need to restart any open Terminal sessions for the change to take effect.
Alternatively, if you have [Homebrew](https://brew.sh/) installed on your Mac, you can install the ballerina package with:

This comment has been minimized.

@samgnaniah

samgnaniah Jun 18, 2018

Contributor

Thank you for this addition. Can you change 'ballerina' to 'Ballerina'? Apart from that this looks good.

@samgnaniah

samgnaniah Jun 18, 2018

Contributor

Thank you for this addition. Can you change 'ballerina' to 'Ballerina'? Apart from that this looks good.

This comment has been minimized.

@btbytes
@btbytes

@TylerJewell TylerJewell self-requested a review Jun 18, 2018

@keizer619

@samgnaniah
We need to check with latest 0.975.0 for homebrew

need to check with latest 0.975.0

@samgnaniah

This comment has been minimized.

Show comment
Hide comment
@samgnaniah

samgnaniah Jun 19, 2018

Contributor

Tested this. It doesn't work for Ballerina 0.975.0 as yet.

Contributor

samgnaniah commented Jun 19, 2018

Tested this. It doesn't work for Ballerina 0.975.0 as yet.

@keizer619

This comment has been minimized.

Show comment
Hide comment
@keizer619

keizer619 Jul 3, 2018

Member

@samgnaniah Currently there is a dependency issue with ballerina, that's why it does not work with 0.975.0. We are going to fix that with 0.980.0.

Member

keizer619 commented Jul 3, 2018

@samgnaniah Currently there is a dependency issue with ballerina, that's why it does not work with 0.975.0. We are going to fix that with 0.980.0.

@rasika

This comment has been minimized.

Show comment
Hide comment
@rasika

rasika Jul 4, 2018

We can merge this now. brew install ballerina will install the latest version(aka. v0.975.1). And for the the future versions it is just a matter of sending a PR to homebrew as Homebrew/homebrew-core#29696

rasika commented Jul 4, 2018

We can merge this now. brew install ballerina will install the latest version(aka. v0.975.1). And for the the future versions it is just a matter of sending a PR to homebrew as Homebrew/homebrew-core#29696

@samgnaniah

This comment has been minimized.

Show comment
Hide comment
@samgnaniah

samgnaniah Jul 13, 2018

Contributor

I'm closing this as it has been open for a bit too long. We will address this once we have the strategy for package managers figured out.

Contributor

samgnaniah commented Jul 13, 2018

I'm closing this as it has been open for a bit too long. We will address this once we have the strategy for package managers figured out.

@samgnaniah samgnaniah closed this Jul 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment