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

terraform: migrate to Homebrew #23585

Merged
merged 1 commit into from
Aug 11, 2016
Merged

Conversation

AnastasiaSulyagina
Copy link
Contributor

Deleting terraform from cask as part of deduplication process . See context here.

Please note that this PR is created to continue case-by-case discussions from thread linked above and if decision is made not to remove the cask, PR will be closed.

cc @MikeMcQuaid

@jawshooah jawshooah merged commit 2b84024 into Homebrew:master Aug 11, 2016
@phinze
Copy link
Contributor

phinze commented Aug 30, 2016

Hello! I read up on the linked issue and I'm here to discuss Terraform as a specific case.

We at HashiCorp currently treat our signed binaries as the official release distribution of our open source software, and while are happy with users building their own versions from source, we have consistently maintained the position that we don't have the bandwidth to officially support the Homebrew Formula.

Given all this, it seems like if there can be only one terraform in the merged namespace, that perhaps the Cask version (which downloads and extracts our official release binaries) should supersede the community-maintained from-source build in the Formula.

My apologies in advance if I'm missing a clear policy or former decision that was agreed upon by the maintainers. Happy to discuss further!

@jawshooah
Copy link
Contributor

Good to see you here, @phinze! The convention so far has been to keep open-source CLI tools in Homebrew, but if HashiCorp would prefer to keep the Cask around instead I'm fine with it.

@MikeMcQuaid, any objections?

@radeksimko
Copy link
Contributor

I think if there is a way to make a Homebrew formula behave like the original cask - i.e. just download the binary and place it to the default path that would be also 👌 (correct me @phinze if I'm wrong).

While I understand why recompilation from scratch may in many cases be useful, the formula upgrade is now blocked purely because a single test is failing while the official binary is available and works just fine.

@MikeMcQuaid
Copy link
Member

@jawshooah I vaguely object but not strongly.

While I understand why recompilation from scratch may in many cases be useful, the formula upgrade is now blocked purely because a single test is failing while the official binary is available and works just fine.

Does it definitely work fine on 10.9? If the installation process is failing there I think that's a valid reason for it to be blocked. It's unclear to me whether that's an indication of a legitimate runtime bug on 10.9.

@cblecker
Copy link
Contributor

cblecker commented Sep 4, 2016

Hi @MikeMcQuaid,
I did a local bottle build under 10.9 using Homebrew/homebrew-core#4237, and terraform builds just fine and creates the bottle. It looks like hashicorp/terraform#8481 was the needed fix, and is patched in to Homebrew/homebrew-core#4237.

That being said, all that might be moot if terraform was moved into homebrew-cask and the pre-compiled and signed binaries are installed into place (instead of compiling locally or installing via bottle).

@MikeMcQuaid
Copy link
Member

@cblecker It's not unreasonable for open-source software to be compiled by packagers rather than the software creators. As mentioned, I'm completely OK with upstream maintaining their own tap, too.

@cblecker
Copy link
Contributor

cblecker commented Sep 4, 2016

@MikeMcQuaid I wasn't suggesting that the current process was unreasonable.. I was only referring to @phinze's above comment that suggested maintaining the cask version over the main homebrew version.

@radeksimko
Copy link
Contributor

FYI: I have just created https://github.com/unofficial-cask/hashicorp-tap in case anyone's interested.

@phinze
Copy link
Contributor

phinze commented Sep 5, 2016

It's not unreasonable for open-source software to be compiled by packagers rather than the software creators.

Totally agreed! The only reason I suggested the binary-download strategy supersede the compile-from-source strategy is because HashiCorp doesn't have the bandwidth to provide consistent support to the community members maintaining the build in the Formula. If there is sufficient community interest to successfully maintain the Formula across Terraform releases, then that's perfectly okay. 👌

@miccal miccal removed the discussion label Dec 23, 2016
@Homebrew Homebrew locked and limited conversation to collaborators May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants