Skip to content

Fixing build problem for Xcode 15#1424

Closed
bombardier200 wants to merge 1 commit intomainfrom
Xcode15Fix
Closed

Fixing build problem for Xcode 15#1424
bombardier200 wants to merge 1 commit intomainfrom
Xcode15Fix

Conversation

@bombardier200
Copy link
Contributor

@bombardier200 bombardier200 commented Sep 19, 2023

Description

Updates Swiftui-windowmanagement to latest version to fix CodeEdit from not building on Xcode15.

Related Issues

Checklist

  • I read and understood the contributing guide as well as the code of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

@matthijseikelenboom
Copy link
Contributor

This would require everyone that is developing on CodeEdit to update to Xcode 15. Do we want to enforce that already?

@bombardier200
Copy link
Contributor Author

I built this on Xcode 14 and it worked?

Copy link
Contributor

@matthijseikelenboom matthijseikelenboom left a comment

Choose a reason for hiding this comment

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

Nvm then, I've said nothing haha

@bombardier200
Copy link
Contributor Author

@matthijseikelenboom I had a caching problem and now it is failing to build with this change on Xcode 14.

@bombardier200 bombardier200 marked this pull request as draft September 19, 2023 20:01
@bombardier200
Copy link
Contributor Author

@Wouter01 do you know the easiest way to keep it building on both Xcode 14 and 15?

@Wouter01
Copy link
Member

@bombardier200 I do not at the moment. But, given that the amount of usage of swift 5.9 exclusive features will increase in the codebase rapidly (swiftlint has new style guides etc) I think requiring Xcode 15 is fine. It also works on Ventura so everyone should be able to upgrade

@austincondiff austincondiff changed the title Fixing build problem for Xcode15 Fixing build problem for Xcode 15 Sep 30, 2023
@austincondiff
Copy link
Collaborator

Where are we on this?

@matthijseikelenboom
Copy link
Contributor

It's a principle decision if everyone should be forced to start using Xcode 15, because if it's merged, you can't use Xcode 14 anymore

@austincondiff
Copy link
Collaborator

But all that has changed was @Wouter01's window package version number. This package version is not compatible with Xcode 14?

@matthijseikelenboom
Copy link
Contributor

It seems like it isn't, no

@Wouter01
Copy link
Member

Wouter01 commented Oct 5, 2023

But all that has changed was @Wouter01's window package version number. This package version is not compatible with Xcode 14?

The package is compatible with Xcode 14. However, the new environment modifier introduced in SwiftUI (https://developer.apple.com/documentation/swiftui/scene/environment(_:_:)) is not available in Xcode 14. A possibility would be to add compiler flags (#if swift version > 5.9 ...), but it's rather unnecessary. @matthijseikelenboom, is there a reason we shouldn't jump to Xcode 15? It's generally not a good idea to mix different Xcode versions (and different swift toolchains)

@matthijseikelenboom
Copy link
Contributor

@Wouter01 Oh I'm not against updating to Xcode 15, but I think it has to be a conscious decision

@austincondiff
Copy link
Collaborator

@lukepistrol would we need an update to our test runner for this? Can you do this?

@lukepistrol
Copy link
Member

@austincondiff I can definitely update to Xcode 15. Just let me know

@austincondiff
Copy link
Collaborator

austincondiff commented Oct 14, 2023

@lukepistrol I updated macOS and downloaded Xcode 15 and put it in the Applications folder. Is there anything else we need to do?

@lukepistrol
Copy link
Member

@lukepistrol I updated macOS and downloaded Xcode 15 and put it in the Applications folder. Is there anything else we need to do?

Maybe just a brew upgrade.

@austincondiff
Copy link
Collaborator

@lukepistrol I did a brew update and brew upgrade and then ran a test. Same result. Any other ideas?

@austincondiff
Copy link
Collaborator

austincondiff commented Oct 18, 2023

Closing because this was necessary and was merged with #1441.

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.

5 participants