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

[xcode15] Add support for Xcode 15 and corresponding OS versions. #18641

Merged
merged 72 commits into from
Aug 8, 2023

Conversation

mandel-macaque
Copy link
Member

No description provided.

rolfbjarne and others added 30 commits August 4, 2023 11:47
Fixes this warning:

> clang: warning: argument unused during compilation: '-stdlib=libc++' [-Wunused-command-line-argument]
…bindings.

The classes have been completely removed, both from the headers and Apple's documentation,
so assume they want it gone and remove it ourselves as well.
The NewsstandKit framework has been completely removed, both from the headers and
Apple's documentation, so assume they want it gone and remove it ourselves as well.
Xcode15 added a new liner, ld_prime which has a number of bugs up to
xcode 15 beta4. We can fix this by fwd the ld64 flag to the linker which
allows to use the OLD ld64 linker over ld_primer until we have a fix
from apple.

Refereces:

Apple docs: https://developer.apple.com/forums/thread/715385
Bug: https://developer.apple.com/forums/thread/733317
@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@@ -28,6 +28,9 @@ public class StringAttributesTests {
[Test]
public void SimpleValuesSet ()
{
if (TestRuntime.CheckXcodeVersion (15, 0))
Assert.Ignore ("Test timeouts on Xcode 15 beta 4.");
Copy link
Member

Choose a reason for hiding this comment

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

Do we have an issue to verify this later?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll add an issue and will link it to the master xcode15 issue

@@ -176,6 +176,10 @@ public static ExecutionResult Execute (string verb, string project, Dictionary<s
// Work around https://github.com/dotnet/msbuild/issues/8845
args.Add ("/v:diag");
args.Add ("/consoleloggerparameters:Verbosity=Quiet");
args.Add ("/consoleloggerparameters:Verbosity=Quiet");
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this argument twice?

Copy link
Member Author

Choose a reason for hiding this comment

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

I bet is a too fast yy.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2023

⚠️ Your code has been reformatted. ⚠️

If this is not desired, add the actions-disable-autoformat label, and revert the reformatting commit.

If files unrelated to your change were modified, try reverting the reformatting commit + merging with the target branch (and push those changes).

@dalexsoto
Copy link
Member

Ref: #18556

@vs-mobiletools-engineering-service2
Copy link
Collaborator

💻 [CI Build] Windows Integration Tests passed 💻

All Windows Integration Tests passed.

Pipeline on Agent
Hash: feb500bdcc4f58fdfd350fe4989767e477d7898e [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 Failed to compare API and create generator diff 🔥

Failed to update apidiff references

Pipeline on Agent
Hash: feb500bdcc4f58fdfd350fe4989767e477d7898e [PR build]

@@ -60,6 +61,7 @@ public NSUserActivity ()
#if __IOS__ && !__MACCATALYST__
public partial class NSUrlConnection {
// Extension from iOS5, NewsstandKit
[EditorBrowsable (EditorBrowsableState.Never)]
Copy link
Contributor

@haritha-mohan haritha-mohan Aug 7, 2023

Choose a reason for hiding this comment

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

this is me just being picky but why not just make all obsolete marked selectors not editor browsable? are there special cases when we would still want this obs selector visible via intellisense?

Copy link
Member Author

Choose a reason for hiding this comment

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

There are some of them that are Obsoleted but not removed, that is, they are a warning that in the future they'll be removed. Others, like this ones, are obsoleted and removed by apple and so we want to make sure that users don't even know about them.

@@ -112,7 +112,9 @@ public void NavigationControllerOverride ()
vc.View.BackgroundColor = UIColor.Green;
};
AppDelegate.PresentModalViewController (nc, 0.5);
if (TestRuntime.CheckXcodeVersion (14, 0)) {
if (TestRuntime.CheckXcodeVersion (15, 0)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

why is the explicit check for Xcode 15 necessary? wasn't the prev logic checking for the same?

Copy link
Member Author

Choose a reason for hiding this comment

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

Look at the content of the if block ;)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

💻 [PR Build] Tests on macOS M1 - Mac Ventura (13.0) passed 💻

All tests on macOS M1 - Mac Ventura (13.0) passed.

Pipeline on Agent
Hash: [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

💻 [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) passed 💻

All tests on macOS M1 - Mac Big Sur (11.5) passed.

Pipeline on Agent
Hash: [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

📚 [PR Build] Artifacts 📚

Packages generated

View packages

Pipeline on Agent
Hash: [PR build]

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🚀 [CI Build] Test results 🚀

Test results

✅ All tests passed on VSTS: simulator tests.

🎉 All 237 tests passed 🎉

Tests counts

✅ bcl: All 69 tests passed. Html Report (VSDrops) Download
✅ cecil: All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests: All 1 tests passed. Html Report (VSDrops) Download
✅ fsharp: All 7 tests passed. Html Report (VSDrops) Download
✅ framework: All 8 tests passed. Html Report (VSDrops) Download
✅ generator: All 2 tests passed. Html Report (VSDrops) Download
✅ interdependent_binding_projects: All 7 tests passed. Html Report (VSDrops) Download
✅ install_source: All 1 tests passed. Html Report (VSDrops) Download
✅ introspection: All 13 tests passed. Html Report (VSDrops) Download
✅ linker: All 65 tests passed. Html Report (VSDrops) Download
✅ mac_binding_project: All 1 tests passed. Html Report (VSDrops) Download
✅ mmp: All 2 tests passed. Html Report (VSDrops) Download
✅ mononative: All 6 tests passed. Html Report (VSDrops) Download
✅ monotouch: All 38 tests passed. Html Report (VSDrops) Download
✅ msbuild: All 2 tests passed. [attempt 2] Html Report (VSDrops) Download
✅ mtouch: All 1 tests passed. Html Report (VSDrops) Download
✅ xammac: All 3 tests passed. Html Report (VSDrops) Download
✅ xcframework: All 8 tests passed. Html Report (VSDrops) Download
✅ xtro: All 2 tests passed. Html Report (VSDrops) Download

Pipeline on Agent
Hash: feb500bdcc4f58fdfd350fe4989767e477d7898e [PR build]

@mandel-macaque mandel-macaque merged commit 8691e43 into xamarin:net8.0-xcode15 Aug 8, 2023
@mandel-macaque mandel-macaque deleted the net8.0-xcode15 branch August 8, 2023 14:28
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