diff --git a/eng/pipelines/runtime.yml b/eng/pipelines/runtime.yml index dddba18e3d1d525e78eb83579a2fae85d79bfd82..882eddc4c2fce33afe1ff503848a2cf80f75bf4e 100644 --- a/eng/pipelines/runtime.yml +++ b/eng/pipelines/runtime.yml @@ -373,7 +373,6 @@ jobs: - Browser_wasm nameSuffix: _EAT runAOT: false - runSmokeOnlyArg: $(_runSmokeTestsOnlyArg) alwaysRun: ${{ variables.isRollingBuild }} # AOT Library tests diff --git a/src/libraries/System.Security.Cryptography/tests/wasm.helix.targets b/eng/testing/helix-extension-example.targets similarity index 100% rename from src/libraries/System.Security.Cryptography/tests/wasm.helix.targets rename to eng/testing/helix-extension-example.targets diff --git a/eng/testing/tests.wasm.targets b/eng/testing/tests.wasm.targets index 398c0b158ded8e38e028d587ce0d2c9d8e6ad903..02af8e04189c71c5fe4ef6025c1503913c29779a 100644 --- a/eng/testing/tests.wasm.targets +++ b/eng/testing/tests.wasm.targets @@ -5,10 +5,6 @@ true $(BundleTestAppTargets);BundleTestWasmApp true - - true - - $(NoWarn);IL2118 $([MSBuild]::NormalizeDirectory($(MonoProjectRoot), 'wasm', 'emsdk')) @@ -33,6 +29,15 @@ <_UseWasmSymbolicator Condition="'$(TestTrimming)' != 'true'">true + + + true + + $(NoWarn);IL2118 + + $(NoWarn);IL2121 + + helix helix diff --git a/eng/testing/workloads-testing.targets b/eng/testing/workloads-testing.targets index 9fc7e7a139161282a2d82de5ada080b0935da4e7..5da047758f81859f7710a60688e856c7187ba655 100644 --- a/eng/testing/workloads-testing.targets +++ b/eng/testing/workloads-testing.targets @@ -116,14 +116,9 @@ - <_DefaultPropsForNuGetBuild Include="Configuration=$(Configuration)" /> - <_DefaultPropsForNuGetBuild Include="TargetOS=Browser" /> - <_DefaultPropsForNuGetBuild Include="TargetArchitecture=wasm" /> - <_DefaultPropsForNuGetBuild Include="ContinuousIntegrationBuild=$(ContinuousIntegrationBuild)" /> - <_NuGetsToBuild Include="$(LibrariesShippingPackagesDir)Microsoft.NETCore.App.Ref.$(_PackageVersion).nupkg" Project="$(InstallerProjectRoot)pkg/sfx/Microsoft.NETCore.App\Microsoft.NETCore.App.Ref.sfxproj" - BuildArgs="@(_DefaultPropsForNuGetBuild -> '-p:%(Identity)', ' ')" + Properties="@(_DefaultPropsForNuGetBuild, ';')" Descriptor="Ref pack"/> @@ -131,10 +126,11 @@ <_PropsForAOTCrossBuild Include="TestingWorkloads=true" /> <_PropsForAOTCrossBuild Include="RuntimeIdentifier=$(NETCoreSdkRuntimeIdentifier)" /> <_PropsForAOTCrossBuild Include="TargetCrossRid=$(RuntimeIdentifier)" /> + <_PropsForAOTCrossBuild Include="DisableSourceLink=true" /> <_NuGetsToBuild Include="$(_AOTCrossNuGetPath)" Project="$(InstallerProjectRoot)pkg/sfx/Microsoft.NETCore.App\Microsoft.NETCore.App.MonoCrossAOT.sfxproj" - BuildArgs="@(_PropsForAOTCrossBuild -> '-p:%(Identity)', ' ')" + Properties="@(_PropsForAOTCrossBuild,';')" Descriptor="AOT Cross compiler"/> @@ -143,7 +139,13 @@ - + + + <_NuGetsToBuild Include="$(LibrariesShippingPackagesDir)Microsoft.NETCore.App.Runtime.Mono.browser-wasm.$(_PackageVersion).nupkg" Project="$(InstallerProjectRoot)pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.Runtime.sfxproj" - BuildArgs="@(_DefaultPropsForNuGetBuild -> '-p:%(Identity)', ' ')" + Properties="@(_DefaultPropsForNuGetBuild, ';')" Dependencies="$(_DefaultRuntimePackNuGetPath)" Descriptor="single threaded runtime pack" Condition="'$(_DefaultBuildVariant)' != '.'" /> diff --git a/src/libraries/System.Security.Cryptography/tests/System.Security.Cryptography.Tests.csproj b/src/libraries/System.Security.Cryptography/tests/System.Security.Cryptography.Tests.csproj index 774292503608e228b029e5177ddedd5794b4b6d3..9b95e193b873a5566f2e0565700c8a2bbcb6c8d9 100644 --- a/src/libraries/System.Security.Cryptography/tests/System.Security.Cryptography.Tests.csproj +++ b/src/libraries/System.Security.Cryptography/tests/System.Security.Cryptography.Tests.csproj @@ -14,8 +14,6 @@ $([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) - wasm.helix.targets - $(WasmXHarnessMonoArgs) --setenv=TEST_EXPECT_SUBTLE_CRYPTO=true $(WasmXHarnessArgs) --web-server-use-cop diff --git a/src/mono/sample/wasm/Directory.Build.props b/src/mono/sample/wasm/Directory.Build.props index 6db0cf52ee3ca1d5b8b792eefbcd84db6a053590..924583ae82d8c0486ca497a9f83f9c02368e4396 100644 --- a/src/mono/sample/wasm/Directory.Build.props +++ b/src/mono/sample/wasm/Directory.Build.props @@ -15,6 +15,7 @@ bin $(MSBuildProjectDirectory)\bin\$(Configuration)\AppBundle\ + $(WasmXHarnessArgs) --web-server-use-cop false diff --git a/src/mono/wasm/Makefile b/src/mono/wasm/Makefile index 6ee499422d591057a653f25ca038257f0c0c9f17..151a7d97d62348e60e355c2b98c2689c3f942ea7 100644 --- a/src/mono/wasm/Makefile +++ b/src/mono/wasm/Makefile @@ -5,6 +5,7 @@ escape_quote = $(subst ",\",$(1)) ifneq ($(V),) MSBUILD_ARGS+=/p:MonoVerboseBuild=true +MSBUILD_ARGS+=-bl endif DOTNET=$(TOP)/dotnet.sh diff --git a/src/tasks/WasmAppBuilder/IcallTableGenerator.cs b/src/tasks/WasmAppBuilder/IcallTableGenerator.cs index 01d7d0eb4479e39a188cf04fa474c184b52ad34b..db530bd57bf826b6286086a2002f98bb52a22a77 100644 --- a/src/tasks/WasmAppBuilder/IcallTableGenerator.cs +++ b/src/tasks/WasmAppBuilder/IcallTableGenerator.cs @@ -152,7 +152,7 @@ private void ProcessType(Type type) } catch (Exception ex) when (ex is not LogAsErrorException) { - Log.LogWarning(null, "WASM0001", "", "", 0, 0, 0, 0, $"Could not get icall, or callbacks for method '{method.Name}' because '{ex.Message}'"); + Log.LogWarning(null, "WASM0001", "", "", 0, 0, 0, 0, $"Could not get icall, or callbacks for method '{type.FullName}::{method.Name}' because '{ex.Message}'"); continue; } diff --git a/src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs b/src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs index 46073228f6777d43b6f0a69b97d034982a43767a..ebc161380594dde0bb3038f99683344cb086254f 100644 --- a/src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs +++ b/src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs @@ -71,7 +71,8 @@ private void CollectPInvokes(List pinvokes, List callb } catch (Exception ex) when (ex is not LogAsErrorException) { - Log.LogWarning(null, "WASM0001", "", "", 0, 0, 0, 0, $"Could not get pinvoke, or callbacks for method '{method.Name}' because '{ex.Message}'"); + Log.LogWarning(null, "WASM0001", "", "", 0, 0, 0, 0, + $"Could not get pinvoke, or callbacks for method '{type.FullName}::{method.Name}' because '{ex.Message}'"); } } @@ -301,7 +302,8 @@ private static bool TryIsMethodGetParametersUnsupported(MethodInfo method, [NotN if (TryIsMethodGetParametersUnsupported(pinvoke.Method, out string? reason)) { - Log.LogWarning(null, "WASM0001", "", "", 0, 0, 0, 0, $"Skipping pinvoke '{pinvoke.Method}' because '{reason}'."); + Log.LogWarning(null, "WASM0001", "", "", 0, 0, 0, 0, + $"Skipping pinvoke '{pinvoke.Method.DeclaringType!.FullName}::{pinvoke.Method}' because '{reason}'."); pinvoke.Skip = true; return null;