-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[dotnet-sdk-9.0.100-rc.2.24474.11] paint.netcore build with error "Missing partial modifier on declaration of type 'AffineTransform2DEffect3'; another partial declaration of this type exists" #43746
Comments
@jaredpar Could you please also take a look at this issue and confirm if this is a RC2 blocker? Thanks. |
cc @Sergio0694 and @manodasanW for triage/analysis on this |
This is the 3rd bug we've seen in this rough setup where we're getting unexpected generation for existing apps. This feels like an issue that we need to be dealing with very quickly. @Sergio0694 we need a higher level of engagement here. |
Thanks for adding the complog @Junjun-zhao. Used that to verify this is not a C# compiler issue. The generated code in this case is not correct compared to the code it generated against (generated code has a |
Will be addressed by microsoft/CsWinRT#1799 |
Thanks @jaredpar @manodasanW. Could I know what release this fix will be merged into? As we are going to sign off the validation for RC2, is it a blocker for it? |
There are two workarounds for this issue:
Once the fix is also available in the Windows SDK projection published to NuGet, someone can also explicitly set |
This is also fixed in the updated Windows SDK projection package. Someone can get the fix to try with the repro before it is available in the .NET SDK by updating the
|
Thanks @manodasanW. We verified the issue with WindowsSdkPackage#10.0.26100.48, it has been fixed. Thank you. |
@manodasanW @marcpopMSFT It still reproduces with the latest build dotnet-sdk-9.0.100-rtm.24515.15 when the default WindowsSdkPackageVersion is 10.0.26100.45. Could you please take a look at this issue again? |
Can you clarify, are you manually setting the windows sdk package version? The fix Manodasan put in was in the windows sdk package and we updated the defaults in the SDK to version .54. If you're overwriting that default back to the old version then it's expected that this would still repro. |
Understand. Thanks @marcpopMSFT This original issue is that failed to build the app when WindowsSdkPackageVersion property was set to .45, so we keep as it-is and reproduced the issue. For the projects which don't not have WindowsSdkPackageVersion property set like Text-Grap project, it will work directly with the latest .NET 9 build# dotnet-sdk-9.0.100-rtm.24515.15 as the defaults in the SDK is version .54 . |
Sounds like it's working now if customers get the right package so closing this out. |
Application Name: paint.netcore
OS: Windows 10 21H2
CPU: X64
.NET Build Number: dotnet-sdk-9.0.100-rc.2.24474.11
App & Source Location checking at:https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2264288
Verify Scenarios:
1). Windows 10 21H2 AMD64 + Default .NET 8 + 9.0.100-rc.1.24452.12 + WindowsSdkPackageVersion 10.0.26100.42: Pass
2). Windows 10 21H2 AMD64 + Default .NET 8 + 9.0.100-rc.1.24452.12 + WindowsSdkPackageVersion 10.0.26100.45: Pass
3). Windows 10 21H2 AMD64 + Default .NET 8 + 9.0.100-rc.2.24474.11 + WindowsSdkPackageVersion 10.0.26100.42: Fail
4). Windows 10 21H2 AMD64 + Default .NET 8 + 9.0.100-rc.2.24474.11 + WindowsSdkPackageVersion 10.0.26100.45: Fail
Description :
When verifying the bug [dotnet-sdk-9.0.100-rc.2.24474.11] paint.netcore build with errors about elements from Microsoft.Windows.SDK.NET are not found · Issue #43707 · dotnet/sdk (github.com), after update the WindowsSdkPackageVersion to 10.0.26100.45 as mentioned in this comment, the errors in that bug are all gone.
But we get a new build error:
App Repro Steps: (You can try it on the repro machine provided in devdiv bug)
The machine has dotnet-sdk-9.0.100-rc.2.24474.11 installed and add its internal feed to Nuget.config file. And the Visual Studio has native libraries like ATL, MFC installed.
1.Copy \SourceShare\paint.net to local machine
2. Open command prompt window and navigate to ...\paint.net folder.
3. Run "
dotnet new globaljson
" to create a global json file and make sure the dotnet-sdk-9.0.100-rc.2.24474.11 assigned:...\paint.net\TargetFramework.props
" file to update the<WindowsSdkPackageVersion>
to10.0.26100.45
:<WindowsSdkPackageVersion>10.0.26100.45</WindowsSdkPackageVersion>
Expected Result:
Build successfully except 2 arm64 related errors that we will ignore them.
Actual Result: (Please get the binlog and compiler log from attachment of devdiv bug), because the size is larger than 25M and can't be uploaded here)
Build failed with errors:
Findings:
All the CS0006 error should be caused by CS0260 error which results to the projects build failed. It is also caused by codegen like #43680.
After clicking the definition from Visual Studio, it shows the "sealed class AffineTransform2DEffect3" is conflicting with source generated "partial class AffineTransform2DEffect3"
Dotnet Info:
@dotnet-actwx-bot @dotnet/compat
The text was updated successfully, but these errors were encountered: