Skip to content
This repository was archived by the owner on Feb 27, 2018. It is now read-only.

Delete even more code and trim down our default message#383

Closed
tianon wants to merge 1 commit intoboot2docker:masterfrom
tianon:moar-deprecation
Closed

Delete even more code and trim down our default message#383
tianon wants to merge 1 commit intoboot2docker:masterfrom
tianon:moar-deprecation

Conversation

@tianon
Copy link
Contributor

@tianon tianon commented Aug 11, 2015

This officially removes support for "download", "init", and "upgrade".

This also adjusts the default message so that it fits comfortably in 80 columns:

$ ./boot2docker download

  WARNING: The 'boot2docker' command line interface is officially deprecated.

  Please switch to Docker Machine (https://docs.docker.com/machine/) ASAP.

  Docker Toolbox (https://docker.com/toolbox) is the recommended install method.

error in run: Sorry, command "download" is no longer supported.

cc @nathanleclaire

This officially removes support for "download", "init", and "upgrade".

This also adjusts the default message so that it fits comfortably in 80 columns:

```console
$ ./boot2docker download

  WARNING: The 'boot2docker' command line interface is officially deprecated.

  Please switch to Docker Machine (https://docs.docker.com/machine/) ASAP.

  Docker Toolbox (https://docker.com/toolbox) is the recommended install method.

error in run: Sorry, command "download" is no longer supported.
```
@tianon
Copy link
Contributor Author

tianon commented Aug 12, 2015

I'm thinking that taking out init might be a bit too far, but I don't see a clean way to do so without totally removing the download code completely, which makes it kind of gimped. 😕

@nathanleclaire
Copy link
Contributor

Argh. Maybe we should just remove upgrade then? download will still download a new ISO but if we're moving to a new repository that shouldn't be an issue, right?

@tianon
Copy link
Contributor Author

tianon commented Aug 12, 2015

Yeah, download will still be reasonably safe.

@tianon
Copy link
Contributor Author

tianon commented Aug 12, 2015

Maybe just the warning is actually enough?

@nathanleclaire
Copy link
Contributor

Yeah, I liked the idea of disabling upgrade completely, but I had forgotten how tightly coupled everything is to cmdDownload.

@tianon
Copy link
Contributor Author

tianon commented Aug 12, 2015

Ok, we tried. 👍

@tianon tianon closed this Aug 12, 2015
@tianon tianon deleted the moar-deprecation branch August 12, 2015 18:15
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.

2 participants