Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

PATH Not Adjusted for All Cases #70

Closed
geigerj0 opened this issue Jan 30, 2025 · 0 comments · Fixed by #71
Closed

PATH Not Adjusted for All Cases #70

geigerj0 opened this issue Jan 30, 2025 · 0 comments · Fixed by #71

Comments

@geigerj0
Copy link
Contributor

geigerj0 commented Jan 30, 2025 •
edited
Loading

PR #50 introduced a change that adjusts PATH to include the devbox binary only in the case of cache misses. However, PATH should always be updated to include ~/.local/bin, regardless of whether there’s a cache hit or miss.

github.heygears.com-hosted runners already include ~/.local/bin in PATH by default, which likely explains why this issue hasn't surfaced yet.

However, for self hosted runners that do not include ~/.local/bin in PATH, the devbox-install-action fails with the following error in case of cache hits as of version v0.12.0:

/__w/_temp/b8c918eb-b7a4-460b-9115-94f59456d885.sh: line 1: devbox: command not found
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(action): Always Add devbox Binary to Path geigerj0/devbox-install-action
1 participant
@geigerj0