Skip to content
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

Fix some issues building for Library Evolution, Xcode 16 #234

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

gemmakbarlow
Copy link

@gemmakbarlow gemmakbarlow commented Sep 29, 2024

  • Fixes some issues building for Library Evolution, Xcode 16
  • Adds Makefile targets used to create the error state prior to fixing them
  • Adds Makefile target build-for-library-evolution to CI

@gemmakbarlow
Copy link
Author

@stephencelis - this should be the last in the series from me 😄

Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Package.swift Outdated Show resolved Hide resolved
[email protected] Outdated Show resolved Hide resolved
@gemmakbarlow
Copy link
Author

I also took a stab at updating CI to address the error from the previous run (not finding the 17.5 simulator with Xcode 16 selected IIRC). Let me know what you think; a couple of ways to do this.

Makefile Outdated Show resolved Hide resolved
@gemmakbarlow
Copy link
Author

Noting that this is working for me locally @stephencelis - might be worth a re-run.

@gemmakbarlow
Copy link
Author

@stephencelis - what do you think about giving this one a re-rerun, given it's working for me locally ? (I don't have that button available in the UI).

@stephencelis
Copy link
Member

@gemmakbarlow I've been re-running but sadly GitHub actions just seem to be flakey. Here's hoping for a green run this time!

@stephencelis
Copy link
Member

@gemmakbarlow Sorry for the churn. I'm frustrated with CI and I think some of the changes I've made to your PR have made things worse in how the simulator is getting resolved. I'm not sure if you want to take another stab at fixing things, or if you want me to try again when I have time.

@gemmakbarlow
Copy link
Author

All good - I can take a stab at fixing it, hopefully later tonight.

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