Sitelet https://github.com/tailscale/github-action/pull/121
Skip to content

Allow to use pre-installed binaries - #121

Closed
borchero wants to merge 11 commits into
tailscale:mainfrom
borchero:no-download
Closed

borchero wants to merge 11 commits into
tailscale:mainfrom
borchero:no-download

Conversation

@borchero

@borchero borchero commented May 1, 2024 •

Copy link
Copy Markdown

Motivation

Potentially, tailscale(d?) have been installed via a separate workflow step to manage the Tailscale version via an external tool (e.g. via conda-forge - tailscale, tailscaled). This change allows tailscale/github-action action to simply use these installed binaries rather than forcing the user to download additional binaries.

@borchero

borchero commented May 3, 2024

Copy link
Copy Markdown
Author

I just noticed that this would also solve #56 (and supersede #103) as brew install tailscale can be run in a separate step.

@borchero

borchero commented May 4, 2024

Copy link
Copy Markdown
Author

FYI, I tested the latest commit in a personal project and it worked nicely.

@jayvdb

jayvdb commented Jun 8, 2024

Copy link
Copy Markdown
Contributor

@borchero , could you rebase this

@borchero

Copy link
Copy Markdown
Author

@jayvdb sorry for the delay, finally done 😄

Comment thread action.yml Outdated
@borchero
borchero requested a review from jayvdb June 17, 2024 20:10
@jayvdb

jayvdb commented Jul 29, 2024 •

Copy link
Copy Markdown
Contributor

ping @knyar & @patrickod & @willnorris

@haron

haron commented Mar 17, 2025

Copy link
Copy Markdown

bump, just in case

@borchero

Copy link
Copy Markdown
Author

I'm not really sure what to do to link an already present binary on Windows -- do I need to do anything if it's on the path?

@oxtoacart

Copy link
Copy Markdown
Collaborator

@borchero Apologies, the GitHub action has since migrated to a JavaScript implementation. If you're willing to submit a new PR against that, we'll review it.

One thing I would recommend changing is to just allow specifying the path to the binaries directly as parameters to the action, that way it doesn't matter if they're on the PATH or not. So instead of a download parameter you would have two parameters, existingTailscalePath and existingTailscaledPath or something like that.

@oxtoacart oxtoacart closed this Oct 20, 2025
@borchero
borchero deleted the no-download branch October 20, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants