search

inlets / inletsctl

Create inlets servers on the top cloud platforms
https://docs.inlets.dev/
MIT License
457 stars 63 forks source link

Fix download command with Apple Silicon machines #122

Closed starbops closed 1 year ago

starbops commented 1 year ago

Signed-off-by: Zespre Schmidt starbops@zespre.com

Description

Related issue: #121

How Has This Been Tested?

I built a dev version of inletsctl and tested on my M1 machine:

$ sudo ./bin/inletsctl-darwin-arm64 download -v
Password:
2023/02/01 16:23:09 https://github.com/inlets/inlets-pro/releases/tag/0.9.13
URL: https://github.com/inlets/inlets-pro/releases/download/0.9.13/inlets-pro-darwin-arm64.
Starting download of inlets-pro 0.9.13, this could take a few moments.
Download completed, make sure that /usr/local/bin is on your path. 
  inlets-pro version

$ /usr/local/bin/inlets-pro version                            
 _       _      _            _
(_)_ __ | | ___| |_ ___   __| | _____   __
| | '_ \| |/ _ \ __/ __| / _` |/ _ \ \ / /
| | | | | |  __/ |_\__ \| (_| |  __/\ V /
|_|_| |_|_|\___|\__|___(_)__,_|\___| \_/

  inlets (tm) - Cloud Native Tunnels
Version: 0.9.13 - 90e5c951334923b4d6d97628be054eb6c39a6170

How are existing users impacted? What migration steps/scripts do we need?

No impact.

Checklist:

I have:

alexellis commented 1 year ago

Thanks for the PR @starbops

The changes are available in: https://github.com/inlets/inletsctl/releases/tag/0.8.20

I still think we should reference arkade which can download inletsctl and use its code / methods instead.