Skip to content

Commit

Permalink
Work around different ownership in container
Browse files Browse the repository at this point in the history
This handles the "dubious ownership" error for the Alpine Linux
container using safe.directory as is done in the Cygwin job.

Another approach may be to actually use a limited user account in
the container, though, and that may be better, since I expect some
of the rmtree tests in test_util.py to fail due to the root user
being able to perform a delete operation the tests assume cannot be
done.
  • Loading branch information
EliahKagan committed Feb 13, 2024
1 parent c7b1fc9 commit b9ceaa4
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/alpine-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ jobs:
with:
fetch-depth: 0

- name: Special configuration for Alpine Linux git
run: |
git config --global --add safe.directory "$(pwd)"
- name: Prepare this repo for tests
run: |
./init-tests-after-clone.sh
Expand Down

0 comments on commit b9ceaa4

Please sign in to comment.