Skip to content

Conversation

@masiuchi
Copy link
Contributor

This pull request fixes two CI errors currently occurring in the master branch.

Update FreeBSD image to 14.3 in Cirrus CI 3ffdaa0

  • freebsd-12-1-release-amd64 image currently in use is no longer available (it appears to have been deleted).

Exclude Perl 5.42 with "strawberry" distribution from GitHub Actions 407986d

@pmqs pmqs merged commit 8cbbd4a into pmqs:master Oct 18, 2025
40 checks passed
@pmqs
Copy link
Owner

pmqs commented Oct 18, 2025

Thanks for the fix @masiuchi . Change merged

@pmqs pmqs self-assigned this Oct 18, 2025
@masiuchi masiuchi deleted the fix/ci-errors branch October 19, 2025 12:19
@masiuchi
Copy link
Contributor Author

Thank you for merging this PR.

@pmqs
Copy link
Owner

pmqs commented Oct 19, 2025

@masiuchi -- getting an error running the modified workflow

[Invalid workflow file: .github/workflows/windows.yml#L1](https://github.com/pmqs/IO-Compress/actions/runs/18617714756/workflow)
(Line: 40, Col: 5): Unexpected value 'exclude'

Aaaah, I see the problem -- the exclude section in your change is indented incorrectly.

I've checked in a fix and run https://github.com/pmqs/IO-Compress/actions/runs/18631075909 has successfully excluded strawberry + 5.42. The fact that the tests fail is another issue.

@masiuchi
Copy link
Contributor Author

@pmqs I apologize for not properly checking the GitHub Actions results.

I thought it was correctly fixed because the errors stopped appearing, but in reality, the Windows workflow had stopped running due to an indentation issue. I'm sorry for the trouble this has caused.
https://github.com/pmqs/IO-Compress/pull/63/checks

I've checked in a fix and run https://github.com/pmqs/IO-Compress/actions/runs/18631075909 has successfully excluded strawberry + 5.42. The fact that the tests fail is another issue.

Thank you for making the corrections. I haven't investigated in detail, but I thought the error message for the 5.40 error seemed similar to the one in the issue below.
StrawberryPerl/Perl-Dist-Strawberry#265

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