Skip to content

Fix curl redirect flag for Psychtoolbox CI download#14

Merged
iandol merged 2 commits into
masterfrom
copilot/fix-matlab-unit-tests
Jun 26, 2026
Merged

Fix curl redirect flag for Psychtoolbox CI download#14
iandol merged 2 commits into
masterfrom
copilot/fix-matlab-unit-tests

Conversation

Copilot AI commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

The "Run MATLAB Unit Tests" CI job was failing because curl wasn't following GitHub's HTTP redirect when downloading the Psychtoolbox zip asset, resulting in a 0-byte file that failed to unzip.

Change

  • .github/workflows/matlab-tests.yml: Add -L flag to curl so it follows the GitHub release redirect to CDN storage
- curl -o /tmp/ptb.zip https://github.com/Psychtoolbox-3/.../3.0.21.0.zip
+ curl -L -o /tmp/ptb.zip https://github.com/Psychtoolbox-3/.../3.0.21.0.zip

GitHub release download URLs respond with a 302 redirect to objects.githubusercontent.com. Without -L, curl writes the redirect response body (empty) to the output file, causing unzip to fail with End-of-central-directory signature not found.

Copilot AI changed the title [WIP] Fix failing GitHub Actions job Run MATLAB Unit Tests Fix curl redirect flag for Psychtoolbox CI download Jun 26, 2026
Copilot AI requested a review from iandol June 26, 2026 11:15
@iandol iandol marked this pull request as ready for review June 26, 2026 11:38
@iandol iandol merged commit 1748b4a into master Jun 26, 2026
1 check failed
@iandol iandol deleted the copilot/fix-matlab-unit-tests branch June 26, 2026 11:57
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

Successfully merging this pull request may close these issues.

2 participants