Fix(actions): Correct sed and wget in macOS generator #129

Closed
mediosdigitalesudem wants to merge 1 commits from fix-macos-generator-action into master
mediosdigitalesudem commented 2025-09-08 22:00:09 +08:00 (Migrated from github.com)

The generator-macos.yml workflow was failing on macOS runners due to two main issues:

  1. The sed commands were using the GNU/Linux -i -e syntax, which is not compatible with the BSD sed found on macOS. This has been corrected to use the sed -i '' -e syntax.

  2. The workflow was fetching patches using hardcoded wget URLs pointing to the original repository. This has been changed to use dynamic GitHub context variables (${{ github.repository }} and ${{ github.ref_name }}) to ensure the workflow uses the patches from the forked repository it is running in.

These changes should resolve the workflow failures and make it more robust for use in forks.

The `generator-macos.yml` workflow was failing on macOS runners due to two main issues: 1. The `sed` commands were using the GNU/Linux `-i -e` syntax, which is not compatible with the BSD `sed` found on macOS. This has been corrected to use the `sed -i '' -e` syntax. 2. The workflow was fetching patches using hardcoded `wget` URLs pointing to the original repository. This has been changed to use dynamic GitHub context variables (`${{ github.repository }}` and `${{ github.ref_name }}`) to ensure the workflow uses the patches from the forked repository it is running in. These changes should resolve the workflow failures and make it more robust for use in forks.

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: 3344/rdgen#129