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

WIP: Add a cask for Docker for Mac #20512

Closed
wants to merge 3 commits into from
Closed

WIP: Add a cask for Docker for Mac #20512

wants to merge 3 commits into from

Conversation

lox
Copy link
Contributor

@lox lox commented Apr 13, 2016

This provides a cask for the new Docker for Mac that is currently in beta release. This is usable now for people with a beta token, but shouldn't be merged until stable versions are released.

Adding a new cask

  • Checked there aren’t open pull requests for the same cask.
  • Checked there aren’t closed issues where that cask was already refused.
  • When naming the cask, followed the token reference.
  • Commit message includes cask’s name.
  • brew cask audit --download docker-for-mac is error-free.
  • brew cask style --fix docker-for-mac left no offenses.
  • brew cask install docker-for-mac worked successfully.
  • brew cask uninstall docker-for-mac worked successfully.

@lox lox changed the title WIP: Add a cask for docker for mac WIP: Add a cask for Docker for Mac Apr 13, 2016
@vitorgalvao
Copy link
Member

You need to fill the PR template. It is not optional.

@vitorgalvao vitorgalvao added the awaiting user reply Issue needs response from a user. label Apr 13, 2016
homepage 'https://beta.docker.com/docs/mac/'
license :gratis

auto_updates false
Copy link
Member

Choose a reason for hiding this comment

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

This is only to be used when the value is true.

@vitorgalvao
Copy link
Member

Thank you for the contribution, but there are a lot of issues with it.

Furthermore, since we can’t actually merge it right now, I’d rather not have PRs lingering indefinitely waiting for an external event beyond our control, particularly since details on the cask can change.

It’s best to just work on the cask when it can actually be merged.

@lox
Copy link
Contributor Author

lox commented Apr 13, 2016

As you wish, there are likely going to be lots of people who are looking for this, I figured it would make sense to have at least something in place as a PR so that effort wasn't duplicated.

@vitorgalvao
Copy link
Member

As you wish, there are likely going to be lots of people who are looking for this, I figured it would make sense to have at least something in place as a PR so that effort wasn't duplicated.

In the PR template there’s a section where you need to confirm you looked to make sure the cask wasn’t submitted before.

@lox
Copy link
Contributor Author

lox commented Apr 13, 2016

Fair enough, I'll revisit when there is a final release and I can address all the points of feedback you raised. Thank you for your time.

@niksy
Copy link
Sponsor Contributor

niksy commented May 30, 2016

Maybe this can be added to https://github.com/caskroom/homebrew-versions in the meantime?

@lox
Copy link
Contributor Author

lox commented May 30, 2016

@niksy I'm unlikely to contribute to cask again, feel free to use whatever code I started on in this PR.

@arbourd
Copy link
Contributor

arbourd commented Jun 14, 2016

For those who came here looking for a versions cask, use a gist to install for now, like this. You can install with brew cask install <link to raw file>.

Thanks for putting this together @lox.

@adidalal adidalal removed the awaiting user reply Issue needs response from a user. label Jul 2, 2016
@axelpale
Copy link

It is a shame that we cannot just brew cask install docker-for-mac

@neverfox
Copy link
Contributor

@axelpale Not sure I understand. You can use brew cask install docker to get Docker for Mac (or docker-beta for the beta version).

@axelpale
Copy link

@neverfox If that is true, great! The earlier replies in this issue left me the feeling that brew cask install docker would install the old docker and not Docker for Mac.

@neverfox
Copy link
Contributor

@axelpale No, but brew install docker (no cask) will install the non-native one.

@axelpale
Copy link

@neverfox Oh, good point. I guess pointing out this distinction will help others who come here for the info about the right installation command. Thanks!

@Tset-Noitamotua
Copy link

aaam ... still confused ... what will I get when I run brew install docker?

a) Docker for Mac
b) Docker Toolbox (old)

image

Regarding brew info docker I guess 'Docker for Mac' will be installed.
What about docker-compose, docker-machine etc.? They are all bundled in recent Docker.dmg. Going with brew I need to install the separately?

Cheers
Tset

@miccal miccal removed the on hold label Dec 23, 2016
@romatthe
Copy link

@Tset-Noitamotua

Currently:

  • brew install docker will simply build from the docker/docker repository. I believe docker-swarm and docker-machine are not included, but there are separate formulas for each of those.
  • brew cask install docker-toolbox will get you Docker Toolbox
  • brew cask install docker will get you Docker for Mac

Essentially, if you want the new native Docker for Mac implementation, you should go for brew cask install docker. Then you'll need to run Docker.app, which will set up the environment for you, including docker, docker-machine and docker-compose.

That's my understanding at least 😅

@Tset-Noitamotua
Copy link

@romatthe
Thanks for clarification. I am running a manual installation for now
but next time will try brew cask install docker.

@HintikkaKimmo
Copy link

Thanks @romatthe , took me some time to figure this out. Even after reading this chain I am not sure why it cannot be called docker for mac

@vitorgalvao
Copy link
Member

vitorgalvao commented Feb 22, 2017

Even after reading this chain I am not sure why it cannot be called docker for mac

Because the token reference must be followed to avoid duplicates, and “for mac” is redundant and nonsensical in most cases.

Yes, some casks may get a bit harder to find, but that’s a more than acceptable tradeoff for a consistent rule that makes most of them better.

@Homebrew Homebrew locked and limited conversation to collaborators May 8, 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