Skip to content

Bundle arm64-ios-simulator libraries with arm64/universal macOS packages#4974

Merged
kinke merged 12 commits intoldc-developers:masterfrom
MrcSnm:patch-1
Sep 27, 2025
Merged

Bundle arm64-ios-simulator libraries with arm64/universal macOS packages#4974
kinke merged 12 commits intoldc-developers:masterfrom
MrcSnm:patch-1

Conversation

@MrcSnm
Copy link
Copy Markdown
Contributor

@MrcSnm MrcSnm commented Aug 25, 2025

Fixes the issue #4972 for the simulator

@kinke
Copy link
Copy Markdown
Member

kinke commented Aug 25, 2025

Thx for trying! You'll need those libs though, adding that as an arm64-specific extra step in https://github.com/ldc-developers/ldc/blob/master/.github/actions/5a-ios/action.yml.

"%%ldcbinarypath%%/../lib-ios-arm64-simulator",
],
rpath = "%%ldcbinarypath%%/../lib-ios-arm64-simulator";
};
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And this section needs to come after the arm64-apple-ios one below. And you should allow for some iOS version in the triple.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that like arm64-apple-ios*-simulator ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.*, this is a regex, see the header comment in some generated ldc2.conf (also wrt. why the simulator section needs to come after the normal iOS one).

@MrcSnm
Copy link
Copy Markdown
Contributor Author

MrcSnm commented Aug 25, 2025

@kinke Can you bump the test runner? Alpine for some reason hang up and stopped my thing on running

@MrcSnm
Copy link
Copy Markdown
Contributor Author

MrcSnm commented Aug 25, 2025

@kinke Got all green and tested the binaries also with my engine to build on the iOS and it worked for me
I've also added some of the integration tests you do for iOS but for the simulator too

@kinke
Copy link
Copy Markdown
Member

kinke commented Aug 26, 2025

Okay thanks! - Let me rebase and slightly rework this (bash supports functions) when I find some time.

@MrcSnm
Copy link
Copy Markdown
Contributor Author

MrcSnm commented Aug 27, 2025

Okay thanks! - Let me rebase and slightly rework this (bash supports functions) when I find some time.

Thanks! I hope we can get that by the next version 👍

Conflicts:
	.github/actions/5a-ios/action.yml
	.github/actions/merge-macos/action.yml
@kinke kinke changed the title Added: arm64-ios-simulator build Bundle arm64-ios-simulator libraries with arm64/universal macOS packages Sep 27, 2025
@kinke kinke merged commit d49d748 into ldc-developers:master Sep 27, 2025
13 of 18 checks passed
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