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

/home/runner/.local/bin missing from PATH #25

Closed
bgilbert opened this issue Jan 16, 2025 · 4 comments
Closed

/home/runner/.local/bin missing from PATH #25

bgilbert opened this issue Jan 16, 2025 · 4 comments

Comments

@bgilbert
Copy link

In the ubuntu-24.04 image, $PATH is:

/snap/bin:/home/runner/.local/bin:/opt/pipx_bin:/home/runner/.cargo/bin:/home/runner/.config/composer/vendor/bin:/usr/local/.ghcup/bin:/home/runner/.dotnet/tools:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin

But in ubuntu-24.04-arm, it is:

/snap/bin:/home/runneradmin/.local/bin:/opt/pipx_bin:/home/runneradmin/.cargo/bin:/home/runneradmin/.config/composer/vendor/bin:/home/runneradmin/.dotnet/tools:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin

pip install (without sudo) installs programs into /home/runner/.local/bin, not /home/runneradmin/.local/bin. Therefore, on ARM only, subsequent workflow steps cannot run programs installed by pip.

@pcolby
Copy link

pcolby commented Jan 17, 2025

The same goes for multiple other paths, such as /home/runneradmin/.local/bin instead of /home/runner/.local/bin, which causes various programs (such as linuxdeploy) to fail, as it tries to inspect those directories (but does not have filesystem permissions to do so).

This relates to #27 too.

@pcolby
Copy link

pcolby commented Jan 17, 2025

Just noting, the works-for-me™ workaround I use is:

  - run: sed -Ee 's/^/PATH=/' -e 's/(runner)admin/\1/g' <<< "${PATH}" | tee -a "$GITHUB_ENV"

This literally just strips the admin from all instances of runneradmin in the $PATH variable.

pcolby added a commit to pcolby/install-linuxdeploy that referenced this issue Jan 17, 2025
pcolby added a commit to pcolby/install-linuxdeploy that referenced this issue Jan 17, 2025
The action already worked without issue on Linux arm64, just the
tests needed to be updated to test the new architecture correctly.

Also included a handy `plugins.md` reference file, which is
intended as an internal reference only.

Squashed commit of the following:

commit 9c09bfc
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 13:11:51 2025 +1100

    Trim leading and trailing whitespace too

commit 33d80a5
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 13:05:59 2025 +1100

    Reduce runs of spaces to single spaces

commit 5f90ee3
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 13:01:36 2025 +1100

    Expect an appimage plugin for Linux arm64

    Even though we didn't request one. Because linuxdeploy supplies
    one itself.

commit 5ba3185
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 12:55:08 2025 +1100

    Show actual and expected plugins

commit b203e15
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 12:45:46 2025 +1100

    Formalise the workaround for Linux arm64 issue #25

    See actions/partner-runner-images#25

commit 839cdde
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 12:34:44 2025 +1100

    Strip erroneous `admin` from $PATH

    See actions/partner-runner-images#25

commit a6b63e5
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 12:18:58 2025 +1100

    Inspect the $PATH

commit ce2cae9
Author: Paul Colby <[email protected]>
Date:   Fri Jan 17 12:16:30 2025 +1100

    Drop version suffix from plugin names filter

commit dab148a
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:18:17 2025 +1100

    Filter plugins **not** supported for this arch

commit 03105ce
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:17:16 2025 +1100

    Filter plugins not supported for this arch

commit dde6cb7
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:15:09 2025 +1100

    Only install Python plugin for x86_64

commit b1c7826
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:12:59 2025 +1100

    Try installing ncurses and python plugins on all arch's

commit 51079c5
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:09:52 2025 +1100

    Use arm64 host for arm64, x86-64 hosts for the rest

commit 0075a95
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:05:30 2025 +1100

    Can execute on arm64 hosts now

commit 81f0f2f
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:03:53 2025 +1100

    Correct workflow syntax

commit bdf0037
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:03:32 2025 +1100

    Execute the correct binary for the current host arch

commit 584887d
Author: Paul Colby <[email protected]>
Date:   Thu Jan 16 18:00:00 2025 +1100

    Try out GitHub's new Linux arm64 runners

    Now in public preview:
    https://github.blog/changelog/2025-01-16-linux-arm64-hosted-runners-now-available-for-free-in-public-repositories-public-preview/
@maxim-lobanov
Copy link
Contributor

Hello 👋
We are aware about this issue and working on it. Fix should be rolled out today.
We will update https://github.com/orgs/community/discussions/148648#discussioncomment-11858717 as soon as it is finished

@bgilbert
Copy link
Author

Looks like this is fixed now. Thanks for the quick response!

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

No branches or pull requests

3 participants