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

Update AppServices Microsoft.Windows.SDK.Contracts package to 10.0.22621.2428 #522

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

michael-hawker
Copy link
Member

@dpaulino was seeing an issue trying to use the AppServices library from a UWP app:

2>C:\Program Files (x86)\Windows Kits\10\bin\10.0.22621.0\XamlCompiler\Microsoft.Windows.UI.Xaml.Common.targets(410,5): XamlCompiler error WMC0901: A type with a full name of 'Windows.UI.Xaml.DependencyObject' already exists in an included reference.

I'm not too sure what happened, but seemed familiar to issues I had faced updating to the latest WASDK when the contracts and stuff weren't also updated alongside. Wonder if some of the updates to our build/tooling have caused a requirement shift here?

Anyway, I tried forcing an update to the project for the Microsoft.Windows.SDK.Contracts package to the latest stable 10.0.22621.2428 and that seemed to resolve the build issue.

Figure updating that in the package here should hopefully resolve the issue without having to declare it in the app?

@dpaulino once this builds, you should be able to try the PR feed one and see if it resolves the issue directly.

@dpaulino
Copy link

@Sergio0694 you mentioned that you found the root cause of the issue. What other changes does this pr need to fix it?

@Sergio0694
Copy link
Member

Sergio0694 commented Jan 27, 2024

Yeah I don't think this PR is going to fix the issue. The NuGet packaging got messed up:

image

You can see we're completely missing the UWP TFM. For comparison, here's what's in 0.0.3 we're using in the Store:

image

Crucially, the UWP TFM doesn't have the package reference to the contracts at all (as it already has WinRT references).

@michael-hawker
Copy link
Member Author

@Arlodotexe what happened here, why is the UWP target missing?

@dpaulino
Copy link

What needs to be done for this to move forward?

@michael-hawker
Copy link
Member Author

What needs to be done for this to move forward?

@dpaulino we've been hung up migrating to .NET 8, that looks like it may be resolved now, and we're still waiting on our other blocking issue for ColorPicker, so Arlo should be able to take a look at this soon and what might have changed here.

@Arlodotexe
Copy link
Member

@Sergio0694 I did a local build and pack of AppServices, it doesn't look like any TFMs are missing at least.

image

I'll run a test to make sure it works as expected in a blank UWP app.

@Sergio0694
Copy link
Member

Yeah that looks good! Does this mean there's something wrong just with the CI build? 🤔

@Arlodotexe
Copy link
Member

Yeah that looks good! Does this mean there's something wrong just with the CI build? 🤔

Here is yesterday's prerelease build pushed to the Labs feed.

image

Yes, it looks like this might be an issue in the CI. I'll poke around and see what I can find.

@Sergio0694 Since this takes a bit of setup in a blank app, do you have a template on hand that you use for testing? Maybe the HelloAppServices project from the announcement post?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

None yet

4 participants