Skip to content

Fix handling MacOS universal package #15

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Dec 21, 2020
Merged

Conversation

jwlawson
Copy link
Owner

@jwlawson jwlawson commented Dec 21, 2020

Starting with 3.19.2, cmake provides a universal package for macos with different file names. This requires some slightly different parsing of the asset information to ensure that these new packages are selected correctly.

Old package name: cmake-3.16.2-Darwin-x86_64.tar.gz
New package name: cmake-3.19.2-macos-universal.tar.gz

This also refactors some of the tests that check the areas touched by these changes, making it easier to reproduce these problems locally on different platforms.

Fixes #13

henryiii and others added 5 commits December 18, 2020 16:43

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Unverified

This user has not yet uploaded their public signing key.

Unverified

This user has not yet uploaded their public signing key.
@jwlawson jwlawson changed the title Fix macos universal Fix macos universal package Dec 21, 2020
@jwlawson jwlawson changed the title Fix macos universal package Fix handling MacOS universal package Dec 21, 2020

Unverified

This user has not yet uploaded their public signing key.
@jwlawson jwlawson merged commit b2390a7 into master Dec 21, 2020
@jwlawson jwlawson deleted the fix_macos_universal branch December 21, 2020 13:59
@jwlawson
Copy link
Owner Author

Fixes #13
Extends on #14

osdeverr pushed a commit to osdeverr/actions-setup-re that referenced this pull request Feb 27, 2025
Starting with 3.19.2, cmake provides a universal package for macos with different file names. This requires some slightly different parsing of the asset information to ensure that these new packages are selected correctly.

Old package name: cmake-3.16.2-Darwin-x86_64.tar.gz
New package name: cmake-3.19.2-macos-universal.tar.gz

This also refactors some of the tests that check the areas touched by these changes, making it easier to reproduce these problems locally on different platforms.

Co-authored-by: Henry Schreiner <HenrySchreinerIII@gmail.com>
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.

Darwin asset missing
2 participants