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

docker-machine-driver-hyperkit: deprecate as it is unmaintained #82122

Merged
merged 3 commits into from
Aug 7, 2021

Conversation

andig
Copy link
Contributor

@andig andig commented Jul 29, 2021

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

@BrewTestBot BrewTestBot added formula deprecated Formula deprecated go Go use is a significant feature of the PR or issue no ARM bottle Formula has no ARM bottle labels Jul 29, 2021
@andig andig mentioned this pull request Jul 29, 2021
@carlocab carlocab changed the title Deprecate docker-machine-driver-hyperkit as it is unmaintained docker-machine-driver-hyperkit: deprecate as it is unmaintained Jul 29, 2021
@carlocab
Copy link
Member

Thanks @andig. Please use Homebrew/core style when writing your commit messages, like so:

docker-machine-driver-hyperkit: deprecate as it is unmaintained

(or similar)

@alebcay
Copy link
Member

alebcay commented Jul 29, 2021

Just curious, do you have any information about this being unmaintained?

@andig
Copy link
Contributor Author

andig commented Jul 29, 2021

Just curious, do you have any information about this being unmaintained?

I've checked the latest commits, see my comment above. Relying on dep in 2021 is unmaintained imho.

docker-machine-driver-hyperkit: deprecate as it is unmaintained

@carlocab could you change these when squashing the commits this time? Will do in future...

@alebcay
Copy link
Member

alebcay commented Jul 29, 2021

Relying on dep in 2021 is unmaintained imho.

Thanks, just wanted to understand your rationale. Since it still seems to be getting a lot of use:

==> Analytics
install: 476 (30 days), 1,467 (90 days), 6,824 (365 days)
install-on-request: 469 (30 days), 1,425 (90 days), 6,640 (365 days)
build-error: 0 (30 days)

I do agree that building with dep is pretty annoying.

@andig
Copy link
Contributor Author

andig commented Jul 29, 2021

dep is deprecated according to golang/go#38158. I did not analyze if we could provide a non-dep build. If that were possible, it would not require deprecation. If not I'd rather deprecate these than not have go 1.17 ;)

@carlocab
Copy link
Member

We should verify first that it is impossible to build this without dep before deprecating it.

@andig
Copy link
Contributor Author

andig commented Jul 29, 2021

Only has Gopkg.toml, no go.mod. One can:

  • go mod init github.com/machine-drivers/docker-machine-driver-hyperkit
  • go mod tidy
  • patch the makefile to get rid of the vendor target
  • ... and make build

but why should we? It is unmaintained and has already caused pain during the go 1.16 cycle.

@SMillerDev
Copy link
Member

but why should we? It is unmaintained and has already caused pain during the go 1.16 cycle.

Because almost 500 people install it every month. Unless there is no way to fix this these people expect Homebrew packages to keep working.

@andig
Copy link
Contributor Author

andig commented Jul 29, 2021

Because almost 500 people install it every month. Unless there is no way to fix this these people expect Homebrew packages to keep working.

I fully understand. However, it is an abandoned product. If people keep expecting it to work they will have to face final breakdown sooner or later. You can keep it half-alive by patching everything in Homebrew or fork the repo and adopt the necessary changes at the root.

I'm personally using Homebrew for simplicity of use. My personal preference would be not fixing outdated stuff where it hinders staying up to date with current releases, unless it can be done upstream.

This is of course personal taste. Happy to have this PR closed for a better solution.

@gromgit
Copy link
Member

gromgit commented Jul 30, 2021

I've pinged upstream to resolve the current situation in an appropriate manner. Let's see what happens.

@SMillerDev
Copy link
Member

I fully understand. However, it is an abandoned product.

So we patch it and mark it deprecated, and next Go release we could drop it completely. Since people will have had some warning.

@andig
Copy link
Contributor Author

andig commented Jul 30, 2021

Will update with patch

@carlocab carlocab merged commit 0603ffb into Homebrew:master Aug 7, 2021
@andig andig deleted the patch-4 branch August 7, 2021 16:18
@github-actions github-actions bot added the outdated PR was locked due to age label Sep 7, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
formula deprecated Formula deprecated go Go use is a significant feature of the PR or issue no ARM bottle Formula has no ARM bottle outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants