ci: add macOS and Windows builds to GitHub Actions, fix LC_ALL warning in Guix container, introduce artifact collection script for GitHub Actions #6421
WINEPREFIX needs to be overridden because of permissions issues with GitHub Actions (build), SC2155 (info) has been suppressed as the expected syntax is clunky and unnecessary (if mktmp fails, wine would refuse to run due to invalid WINEPREFIX, which would trigger an error).
The HOST check before running setup_sdk.sh isn't a part of the shell script itself as the script is written to not have to rely on external variables being set, so the caller is expected to know the conditions it'll impose to run setup_sdk.sh, the script itself remaining agnostic.
guix-start doesn't benefit from deduplication because the scripts there are expected to be self-contained and therefore, cannot link back to this script as it will not be present in the container.
Breaking Changes
None expected.
Checklist
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas (note: N/A)
[x] I have added or updated relevant unit/integration/functional/e2e tests (note: N/A)
[x] I have made corresponding changes to the documentation (note: N/A)
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)
Additional Information
WINEPREFIX
needs to be overridden because of permissions issues with GitHub Actions (build), SC2155 (info) has been suppressed as the expected syntax is clunky and unnecessary (ifmktmp
fails,wine
would refuse to run due to invalidWINEPREFIX
, which would trigger an error).HOST
check before runningsetup_sdk.sh
isn't a part of the shell script itself as the script is written to not have to rely on external variables being set, so the caller is expected to know the conditions it'll impose to runsetup_sdk.sh
, the script itself remaining agnostic.guix-start
doesn't benefit from deduplication because the scripts there are expected to be self-contained and therefore, cannot link back to this script as it will not be present in the container.Breaking Changes
None expected.
Checklist