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

(macOS pkg for) iOS workload doesn't include sentinel #67

Closed
chucker opened this issue Feb 23, 2021 · 2 comments
Closed

(macOS pkg for) iOS workload doesn't include sentinel #67

chucker opened this issue Feb 23, 2021 · 2 comments

Comments

@chucker
Copy link
Contributor

chucker commented Feb 23, 2021

(Related: #47)

I'm not sure if this is by design, but the Android workload adds a .sentinel file to ./sdk/, and the iOS one doesn't:

image

Consequently, only installing the SDK and the iOS workload will result in the "The target platform identifier ios was not recognized" error.

If this is intended, the README should make it clearer that building the iOS sample on macOS will fail unless you also install the Android workload.

@jonathanpeppers
Copy link
Member

We are hoping the need for a .sentinel file at all can go away.

The dotnet/sdk did this so they could ship workload support behind a feature flag in .NET 5. I think they can get rid of it soon.

@jonathanpeppers
Copy link
Member

In .NET 6 Preview 4, EnableWorkloadResolver.sentinel is no longer needed.

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

No branches or pull requests

2 participants