| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
@Sergio0694 Would it be possible to validate the fix by building and testing Windows Store app with NAOT from main?
|
Sorry, something went wrong.
|
So I managed to get the Store to run on .NET 10 CoreCLR, but ILC is not happy 😅 Perhaps it would be simpler to just test with an SDK build from this branch? Is there a pipeline I can run to get it? |
Sorry, something went wrong.
|
Weird, this overload was added in #110234, this must be using some AotSdk with an old corelib (and likely old runtime too, without your fix). What are your repro steps? If you download the latest .NET 10 SDK, it should already have the ILCompiler with your fix. So all that should be needed is to put it in the PATH, add NuGet.config with the .NET 10 feed, update Store to target .NET 10 and that's it. |
Sorry, something went wrong.
|
That's exactly what I did 🥲 |
Sorry, something went wrong.
|
Update: I tested again (this time in Release, last time I was trying to run in Debug + Native AOT), and it works! 🎉 I can no longer repro the hangs (no spinning circle on the mouse cursor, no hangs in event viewer), and I can close and reopen the Store multiple times in rapid succession without getting stuck at the OS splash screen (due to the previous instance being hung)! |
Sorry, something went wrong.
|
@Sergio0694 Thank you for testing the fix! I have submitted the change into servicing approval process. |
Sorry, something went wrong.
There was a problem hiding this comment.
lgtm. we will take for consideration in 9.0.x
Sorry, something went wrong.
|
@jkotas please take a look at the PR failures and merge when ready |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Backport of #110551 to release/9.0-staging
/cc @jkotas @Sergio0694
Customer Impact
Intermittent hang when UWP/WinUI app is suspended. Reported by Windows Store.
Regression
This is .NET Native -> .NET 9 regression on UWP. The fix updates IReferenceTrackerHost::ReleaseDisconnectedReferenceSources implementation in .NET 9 to match .NET Native (ie reverts the offending change).
Testing
Fix validated on private build by Windows Store
Risk
Low