-
Notifications
You must be signed in to change notification settings - Fork 130
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
Fix installation error on self-hosted runner #138
base: master
Are you sure you want to change the base?
Fix installation error on self-hosted runner #138
Conversation
149c29e
to
6a81b92
Compare
b5a8507
to
97d4517
Compare
97d4517
to
737ef29
Compare
updates from origin
ef58b4a
to
48214d1
Compare
This works, however, I think that the download should not depend on the username (we use "root" on all our runners) but rather on the runner name instead. |
Yes true. Good point. Let me fix it. |
🤔 actually... it seems there is no (nice) way how to get the runner name from JS action actions/toolkit#1681 |
also waiting for the merge on this :-) |
Here's an idea: why not use I've also opened a discussion here https://github.com/orgs/community/discussions/137836 if you care to upvote :) |
@simonjur https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables#default-environment-variables it seems that |
@soulseekah many thanks for the suggestion 👍 . I was a bit stuck on this. Let me try it and get back to you (hopefully today) |
just a note to this solution. yep this would work and I was considering it...
But it's a good suggestion and a last-resort fallback maybe... we'll see 🙈 |
yep env var is there
and like this on gh-hosted runner:
let me finish this PR then... |
fixed ✅ |
…running and share temp directory
8bd842f
to
978f6e0
Compare
Suffix download temporary file with version. Delete the temporary file after extraction.
Should fix #127