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

hma-vpn: rename from hma-pro-vpn #169298

Merged
merged 3 commits into from
Mar 19, 2024
Merged

hma-vpn: rename from hma-pro-vpn #169298

merged 3 commits into from
Mar 19, 2024

Conversation

daeho-ro
Copy link
Member

It seems that the pro version is not supported anymore.
https://github.com/Homebrew/homebrew-cask/blob/8166f6d33303cf75a0105c44bc46dce96729987a/Casks/h/hma-pro-vpn.rb

So, I add the generic version of hma-vpn.

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • [v] Named the cask according to the token reference.
  • [v] Checked the cask was not already refused.
  • [v] Checked the cask is submitted to the correct repo.
  • [v] brew audit --cask --new <cask> worked successfully.
  • [v] HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • [v] brew uninstall --cask <cask> worked successfully.

@daeho-ro daeho-ro changed the title add hma-vpn hma-vpn latest (new cask) Mar 18, 2024
@bevanjkay
Copy link
Member

Is this the same product, but renamed?

@daeho-ro
Copy link
Member Author

@bevanjkay
I think the pro version is no more maintained and I honestly cannot check that in more details. Anyway, the product is now given by hma-vpn and the package file has different checksum and I decide to register it as a new cask. Also there was a request to check that is going on the pro version from the discussion, see https://github.com/orgs/Homebrew/discussions/5177.

@bevanjkay
Copy link
Member

I can see that in 2018 there is reference on the same download page to HMA Pro VPN, so I think it is safe to assume that this is the same app, just with a different name.
https://web.archive.org/web/20180929092636/https://www.hidemyass.com/en-au/download-vpn-mac

@daeho-ro
Copy link
Member Author

Then should I update the existing one not add the new one, right? I am ready, just confirm please.

@krehel
Copy link
Member

krehel commented Mar 19, 2024

I would agree with the rename, not new.

@daeho-ro daeho-ro changed the title hma-vpn latest (new cask) hma-pro-vpn latest Mar 19, 2024
@krehel krehel added the awaiting user reply Issue needs response from a user. label Mar 19, 2024
@daeho-ro
Copy link
Member Author

@krehel , Is this correct way to rename a package? I am not sure about this.

Signed-off-by: Justin Krehel <[email protected]>
@krehel
Copy link
Member

krehel commented Mar 19, 2024

The goal is to update and rename hma-pro-vpn to hma-vpn? If so, then the changes I've pushed should address that.

Signed-off-by: Justin Krehel <[email protected]>
@krehel krehel changed the title hma-pro-vpn latest hma-vpn: rename from hma-pro-vpn Mar 19, 2024
@daeho-ro
Copy link
Member Author

That is right. I missed that!

@krehel
Copy link
Member

krehel commented Mar 19, 2024

No worries @daeho-ro - happy to help. Thanks for taking the initiative to open this and get it moving forward.

@krehel krehel removed the awaiting user reply Issue needs response from a user. label Mar 19, 2024
@p-linnane p-linnane merged commit 78b8cd7 into Homebrew:master Mar 19, 2024
12 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants