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
[Blazor] Fix concurrency problem in Android builds #19140
Conversation
* Disable static web assets on nested Rid builds as its not necessary it runs there. * The outer build already collects the assets and transforms them into Maui assets.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This actually makes sense. I'm not sure why we never thought to do it this way!
src/BlazorWebView/src/Maui/build/Microsoft.AspNetCore.Components.WebView.Maui.props
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems it's still failing
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-x86\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\Controls\samples\Controls.Sample\Maui.Controls.Sample.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-arm64\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\Controls\samples\Controls.Sample\Maui.Controls.Sample.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-x64\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\Controls\samples\Controls.Sample\Maui.Controls.Sample.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-arm\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\Controls\samples\Controls.Sample\Maui.Controls.Sample.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-x86\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\BlazorWebView\tests\MauiDeviceTests\MauiBlazorWebView.DeviceTests.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-x64\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\BlazorWebView\tests\MauiDeviceTests\MauiBlazorWebView.DeviceTests.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-arm\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\BlazorWebView\tests\MauiDeviceTests\MauiBlazorWebView.DeviceTests.csproj::TargetFramework=net8.0-android]
D:\a\_work\1\s\bin\dotnet\sdk\8.0.100\Sdks\Microsoft.NET.Sdk.StaticWebAssets\targets\Microsoft.NET.Sdk.StaticWebAssets.References.targets(16,5): error : Manifest file at 'obj\Release\net8.0-android\android-arm64\staticwebassets.build.json' not found. [D:\a\_work\1\s\src\BlazorWebView\tests\MauiDeviceTests\MauiBlazorWebView.DeviceTests.csproj::TargetFramework=net8.0-android]
The test was not importing the props file. Hopefully the latest change will address it |
This should also close this #11872 |
dotnet/aspnetcore#52460