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

Set default features flags to match to iOS SDK defaults #49635

Merged
merged 8 commits into from
Mar 24, 2021

Conversation

MaximLipnin
Copy link
Contributor

@MaximLipnin MaximLipnin commented Mar 15, 2021

Contributes to #43865.

Based on https://github.com/xamarin/xamarin-macios/blob/834b0888853210ca01d5e8f5ce2c65fe9c55fc10/dotnet/targets/Xamarin.Shared.Sdk.targets#L138-L146

The changes:

  • added a property group with iOS/iOSSimulator default feature flags;
  • aligned the conditions to check the properties activating the defaults for true in order to follow better msbuild practice.

@ghost
Copy link

ghost commented Mar 15, 2021

Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

Contributes to #43865.

Based on https://github.com/xamarin/xamarin-macios/blob/834b0888853210ca01d5e8f5ce2c65fe9c55fc10/dotnet/targets/Xamarin.Shared.Sdk.targets#L138-L146

Author: MaximLipnin
Assignees: -
Labels:

area-Infrastructure-libraries

Milestone: -

@MaximLipnin MaximLipnin force-pushed the add_ios_default_feature_flags branch 3 times, most recently from ae6b1b4 to 02e24a1 Compare March 22, 2021 07:10
Copy link
Member

@steveisok steveisok left a comment

Choose a reason for hiding this comment

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

LGTM. I'd rebase and run one more build before merging.

@MaximLipnin MaximLipnin force-pushed the add_ios_default_feature_flags branch from bd7a660 to db62e3a Compare March 23, 2021 08:24
eng/testing/tests.mobile.targets Outdated Show resolved Hide resolved
eng/testing/tests.mobile.targets Outdated Show resolved Hide resolved
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
@MaximLipnin MaximLipnin merged commit 78ae1d6 into dotnet:main Mar 24, 2021
@MaximLipnin MaximLipnin deleted the add_ios_default_feature_flags branch March 24, 2021 08:06
@ghost ghost locked as resolved and limited conversation to collaborators Apr 23, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants