Skip to content

Commit

Permalink
Merge pull request #1709 from EliahKagan/cygwin-ci-path
Browse files Browse the repository at this point in the history
Run Cygwin CI workflow commands in login shells
  • Loading branch information
Byron committed Oct 14, 2023
2 parents fee2444 + e8956e5 commit 97c59ae
Showing 1 changed file with 15 additions and 9 deletions.
24 changes: 15 additions & 9 deletions .github/workflows/cygwin-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,30 +10,36 @@ jobs:
fail-fast: false

env:
CHERE_INVOKING: 1
TMP: "/tmp"
TEMP: "/tmp"
CHERE_INVOKING: "1"
CYGWIN_NOWINPATH: "1"

defaults:
run:
shell: C:\cygwin\bin\bash.exe --noprofile --norc -exo pipefail -o igncr "{0}"
shell: C:\cygwin\bin\bash.exe --login --norc -eo pipefail -o igncr "{0}"

steps:
- name: Force LF line endings
run: |
git config --global core.autocrlf false # Affects the non-Cygwin git.
shell: bash
shell: bash # Use Git Bash instead of Cygwin Bash for this step.

- uses: actions/checkout@v4
with:
fetch-depth: 0
submodules: recursive

- uses: cygwin/cygwin-install-action@v4
- name: Install Cygwin
uses: cygwin/cygwin-install-action@v4
with:
packages: python39 python39-pip python39-virtualenv git
add-to-path: false # No need to change $PATH outside the Cygwin environment.

- name: Special configuration for Cygwin's git
- name: Arrange for verbose output
run: |
# Arrange for verbose output but without shell environment setup details.
echo 'set -x' >~/.bash_profile
- name: Special configuration for Cygwin git
run: |
git config --global --add safe.directory "$(pwd)"
git config --global core.autocrlf false
Expand All @@ -57,7 +63,7 @@ jobs:
- name: Install project and test dependencies
run: |
python -m pip install ".[test]"
pip install ".[test]"
- name: Show version and platform information
run: |
Expand All @@ -71,4 +77,4 @@ jobs:
- name: Test with pytest
run: |
python -m pytest --color=yes -p no:sugar --instafail -vv
pytest --color=yes -p no:sugar --instafail -vv

0 comments on commit 97c59ae

Please sign in to comment.