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

shell: bash steps continue to use the system perl #328

Closed
skirmess opened this issue Aug 28, 2020 · 3 comments
Closed

shell: bash steps continue to use the system perl #328

skirmess opened this issue Aug 28, 2020 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@skirmess
Copy link
Contributor

On Windows, if we use shell: bash the PATH env variable always starts with /mingw64/bin and /usr/bin.

  job:
    runs-on: windows-latest
    steps:
      - uses: shogo82148/actions-setup-perl@v1

      # uses powershell and runs with the Perl from actions-setup-perl
      - run: perl -V

      # uses bash and runs with the system Perl
      - run: perl -V
        shell: bash

I can't see a way to change this behavior as the /usr/bin directory seems to be hard coded somehow and I'm not sure if deleting /usr/bin/perl.exe on Windows is a good idea? If you don't see a way to fix the path we should document that the bash shell is not supported under Windows.

@shogo82148
Copy link
Owner

Thank you for your report.
I seems that bash on Windows automatically adds /usr/bin into the PATH environment value.

I'm not sure if deleting /usr/bin/perl.exe on Windows is a good idea?

deleting /usr/bin/perl.exe on Windows is a bad idea. Because Git on Windows uses perl.
It may break the actions/checkout action.

Unfortunately, I have no idea...
I'll document this issue.

@shogo82148 shogo82148 added the help wanted Extra attention is needed label Aug 29, 2020
shogo82148 added a commit that referenced this issue Aug 29, 2020
shogo82148 added a commit that referenced this issue Aug 29, 2020
* add a link to #328

* add a link to #225
shogo82148 added a commit that referenced this issue Sep 7, 2020
#328 Document which shell are supported
@skirmess
Copy link
Contributor Author

Thanks

@hurricup
Copy link

hurricup commented Mar 6, 2023

Would be also nice to add path to the perl containing directory to the action outputs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants