Skip to content

[Intents] Fix watchOS INCarSeatResolutionResult breaking changes - #2690

Merged
spouliot merged 2 commits into
dotnet:xcode9from
dalexsoto:intentsgm
Sep 14, 2017
Merged

[Intents] Fix watchOS INCarSeatResolutionResult breaking changes#2690
spouliot merged 2 commits into
dotnet:xcode9from
dalexsoto:intentsgm

Conversation

@dalexsoto

Copy link
Copy Markdown
Member

INCarSeatResolutionResult is not available in watchOS, unfortunately
it was bound pre xcode9 so until XAMCORE_4_0 happens it won't get removed

Type Changed: Intents.INCarSeatResolutionResult

Removed methods:

    public static INCarSeatResolutionResult GetConfirmationRequired (INCarSeat valueToConfirm);
    public static INCarSeatResolutionResult GetSuccess (INCarSeat resolvedValue);

INCarSeatResolutionResult is not available in watchOS, unfortunately
it was bound pre xcode9 so until XAMCORE_4_0 happens it wont get removed

Type Changed: Intents.INCarSeatResolutionResult

Removed methods:

    public static INCarSeatResolutionResult GetConfirmationRequired (INCarSeat valueToConfirm);
    public static INCarSeatResolutionResult GetSuccess (INCarSeat resolvedValue);
Comment thread src/intents.cs Outdated
#if XAMCORE_4_0
[NoWatch]
#else
[Obsoleted (PlatformName.WatchOS, message:"'INCarSeatResolutionResult' is not available in watchOS.")]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

did Apple obsolete it ? or was it a binding mistake ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

rdar was filed, let's just remove the XAMCORE_4_0 (both sides)

@monojenkins

Copy link
Copy Markdown
Contributor

Build success

@monojenkins

Copy link
Copy Markdown
Contributor

Build success

@spouliot
spouliot merged commit 274a34c into dotnet:xcode9 Sep 14, 2017
@dalexsoto
dalexsoto deleted the intentsgm branch September 14, 2017 05:16
mandel-macaque added a commit to mandel-macaque/xamarin-macios that referenced this pull request Aug 23, 2023
Bump maccore to bing the changes for governance and some others:

* Merge 'xcode15' into 'main'. (dotnet#2690)
* [Submission] Add NotaryTool Task (dotnet#2689)
* [XmlDocSync] Remove the XmlDocSync reference from the project. (dotnet#2693)
* [mlaunch] Refactor project structure (dotnet#2694)
* [certificates] Update la_dev_apple.p12, la_distr_apple.p12 and la_mac_installer_distr.p12 (dotnet#2697)
* [Governance] Update the false positives files based on the last runs. (dotnet#2698)

Full idff: https://github.com/xamarin/maccore/compare/fd15821e6780a7439df467c69eb511a046adfa06...63fd14954ca3328219c5f79c754f0b329091535c
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.

4 participants