From 4038554122fa42698ffdad0adbb9158d5f3fe2d3 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:27:13 -0400 Subject: [PATCH 1/8] [wasm] WasmAppBuilder: detect multiple paths mapping to the same vfs .. path. This shows up at runtime as a cryptic `FS Error`, because it fails to create the parent directory. --- src/tasks/AotCompilerTask/MonoAOTCompiler.cs | 1 - src/tasks/WasmAppBuilder/WasmAppBuilder.cs | 17 +++++++++++++++++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/src/tasks/AotCompilerTask/MonoAOTCompiler.cs b/src/tasks/AotCompilerTask/MonoAOTCompiler.cs index e8c3680dd6e4f..dee6b537b7bd5 100644 --- a/src/tasks/AotCompilerTask/MonoAOTCompiler.cs +++ b/src/tasks/AotCompilerTask/MonoAOTCompiler.cs @@ -395,7 +395,6 @@ private bool ExecuteInternal() new ParallelOptions { MaxDegreeOfParallelism = allowedParallelism }, (args, state) => PrecompileLibraryParallel(args, state)); - Log.LogMessage(MessageImportance.High, $"result: {result.IsCompleted}"); if (result.IsCompleted) { int numUnchanged = _totalNumAssemblies - _numCompiled; diff --git a/src/tasks/WasmAppBuilder/WasmAppBuilder.cs b/src/tasks/WasmAppBuilder/WasmAppBuilder.cs index e356291d83a31..4c3054cad30e9 100644 --- a/src/tasks/WasmAppBuilder/WasmAppBuilder.cs +++ b/src/tasks/WasmAppBuilder/WasmAppBuilder.cs @@ -3,6 +3,7 @@ using System; using System.Collections.Generic; +using System.Collections.Specialized; using System.Diagnostics.CodeAnalysis; using System.IO; using System.Linq; @@ -222,6 +223,7 @@ private bool ExecuteInternal () Directory.CreateDirectory(supportFilesDir); var i = 0; + StringDictionary targetPathTable = new(); foreach (var item in FilesToIncludeInFileSystem) { string? targetPath = item.GetMetadata("TargetPath"); @@ -232,6 +234,21 @@ private bool ExecuteInternal () // We normalize paths from `\` to `/` as MSBuild items could use `\`. targetPath = targetPath.Replace('\\', '/'); + if (targetPathTable.ContainsKey(targetPath)) + { + string firstPath = Path.GetFullPath(targetPathTable[targetPath]!); + string secondPath = Path.GetFullPath(item.ItemSpec); + + if (firstPath == secondPath) + { + Log.LogWarning($"Found identical vfs mappings for target path: {targetPath}, source file: {firstPath}. Ignoring."); + continue; + } + + throw new LogAsErrorException($"Found more than one file mapping to the target VFS path: {targetPath}. Source files: {firstPath}, and {secondPath}"); + } + + targetPathTable[targetPath] = item.ItemSpec; var generatedFileName = $"{i++}_{Path.GetFileName(item.ItemSpec)}"; From 80238550267a115785c4ab5d17bf8348fba6838b Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:29:28 -0400 Subject: [PATCH 2/8] [wasm] Add new DisableParallelEmccCompile property --- src/mono/wasm/build/WasmApp.Native.targets | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/mono/wasm/build/WasmApp.Native.targets b/src/mono/wasm/build/WasmApp.Native.targets index 69facc5d7204a..8476977afef42 100644 --- a/src/mono/wasm/build/WasmApp.Native.targets +++ b/src/mono/wasm/build/WasmApp.Native.targets @@ -152,6 +152,8 @@ <_WasmPInvokeTablePath>$(_WasmIntermediateOutputPath)pinvoke-table.h <_WasmPInvokeHPath>$(_WasmRuntimePackIncludeDir)wasm\pinvoke.h <_DriverGenCPath>$(_WasmIntermediateOutputPath)driver-gen.c + false + $(DisableParallelAot) <_DriverGenCNeeded Condition="'$(_DriverGenCNeeded)' == '' and '$(_WasmShouldAOT)' == 'true'">true @@ -283,7 +285,7 @@ - + <_WasmSourceFileToCompile Remove="@(_WasmSourceFileToCompile)" /> <_WasmSourceFileToCompile Include="@(_WasmRuntimePackSrcFile)" Dependencies="%(_WasmRuntimePackSrcFile.Dependencies);$(_EmccDefaultFlagsRsp);$(_EmccCompileRsp)" /> @@ -292,6 +294,7 @@ SourceFiles="@(_WasmSourceFileToCompile)" Arguments='"@$(_EmccDefaultFlagsRsp)" "@$(_EmccCompileRsp)"' EnvironmentVariables="@(EmscriptenEnvVars)" + DisableParallelCompile="$(DisableParallelEmccCompile)" OutputMessageImportance="$(_EmccCompileOutputMessageImportance)"> @@ -308,11 +311,12 @@ <_BitCodeFile Dependencies="%(_BitCodeFile.Dependencies);$(_EmccDefaultFlagsRsp);$(_EmccCompileBitcodeRsp)" /> - + @@ -371,7 +375,7 @@ DependsOnTargets="_WasmSelectRuntimeComponentsForLinking;_WasmCompileAssemblyBitCodeFilesForAOT;_WasmWriteRspFilesForLinking" Returns="@(FileWrites)" > - + From e51a76f272f7dce1cd65e801efe5c237eb243c3b Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:34:21 -0400 Subject: [PATCH 3/8] [wasm] Don't include satellite assemblies twice for aot/eat case --- eng/testing/tests.wasm.targets | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/testing/tests.wasm.targets b/eng/testing/tests.wasm.targets index f68b4f9889a79..b41143fd4a76d 100644 --- a/eng/testing/tests.wasm.targets +++ b/eng/testing/tests.wasm.targets @@ -159,7 +159,7 @@ -1 - + <_SatelliteAssemblies Include="$(PublishDir)*\*.resources.dll" /> <_SatelliteAssemblies CultureName="$([System.IO.Directory]::GetParent('%(Identity)').Name)" /> <_SatelliteAssemblies TargetPath="%(CultureName)\%(FileName)%(Extension)" /> From 8436395924bd9e1341197e04e4394f3974a4e6da Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:36:12 -0400 Subject: [PATCH 4/8] [wasm] add $(IsHighAotMemoryUsageTest) to use different optimization options for avoiding OOM .. and use a separate script template for AOT builds. --- eng/testing/WasmRunnerAOTTemplate.sh | 57 ++++++++++++++++++++++++++++ eng/testing/tests.targets | 3 +- eng/testing/tests.wasm.targets | 27 +++++++++---- 3 files changed, 78 insertions(+), 9 deletions(-) create mode 100644 eng/testing/WasmRunnerAOTTemplate.sh diff --git a/eng/testing/WasmRunnerAOTTemplate.sh b/eng/testing/WasmRunnerAOTTemplate.sh new file mode 100644 index 0000000000000..5eb2d76d0774b --- /dev/null +++ b/eng/testing/WasmRunnerAOTTemplate.sh @@ -0,0 +1,57 @@ +#!/usr/bin/env bash + +EXECUTION_DIR=$(dirname $0) +SCENARIO=$3 + +cd $EXECUTION_DIR + +if [ -z "$HELIX_WORKITEM_UPLOAD_ROOT" ]; then + XHARNESS_OUT="$EXECUTION_DIR/xharness-output" +else + XHARNESS_OUT="$HELIX_WORKITEM_UPLOAD_ROOT/xharness-output" +fi + +if [ ! -z "$XHARNESS_CLI_PATH" ]; then + # When running in CI, we only have the .NET runtime available + # We need to call the XHarness CLI DLL directly via dotnet exec + HARNESS_RUNNER="dotnet exec $XHARNESS_CLI_PATH" +else + HARNESS_RUNNER="dotnet xharness" +fi + +if [ "$SCENARIO" == "WasmTestOnBrowser" ]; then + XHARNESS_COMMAND="test-browser" +elif [ -z "$XHARNESS_COMMAND" ]; then + XHARNESS_COMMAND="test" +fi + +function BuildAOT() +{ + local projectFile=$1 + local binLog=$2 + shift 2 + + time dotnet msbuild $projectFile /bl:$binLog $* + local buildExitCode=$? + + echo "\n** Performance summary for the build **\n" + dotnet msbuild $binLog -clp:PerformanceSummary -v:q -nologo + if [[ "$(uname -s)" == "Linux" && $buildExitCode -ne 0 ]]; then + echo "\nLast few messages from dmesg:\n" + dmesg | tail -n 20 + fi + + echo + echo + + return $buildExitCode +} + +# RunCommands defined in tests.mobile.targets +[[RunCommands]] + +_exitCode=$? + +echo "XHarness artifacts: $XHARNESS_OUT" + +exit $_exitCode diff --git a/eng/testing/tests.targets b/eng/testing/tests.targets index 1e7ec3b173dc3..42e8e54534db5 100644 --- a/eng/testing/tests.targets +++ b/eng/testing/tests.targets @@ -4,7 +4,8 @@ RunnerTemplate.sh AppleRunnerTemplate.sh AndroidRunnerTemplate.sh - WasmRunnerTemplate.sh + WasmRunnerAOTTemplate.sh + WasmRunnerTemplate.sh WasmRunnerTemplate.cmd diff --git a/eng/testing/tests.wasm.targets b/eng/testing/tests.wasm.targets index b41143fd4a76d..1554237bf38ff 100644 --- a/eng/testing/tests.wasm.targets +++ b/eng/testing/tests.wasm.targets @@ -38,12 +38,12 @@ - <_AOTBuildCommand>dotnet msbuild publish/ProxyProjectForAOTOnHelix.proj /bl:$XHARNESS_OUT/AOTBuild.binlog + <_AOTBuildCommand>BuildAOT publish/ProxyProjectForAOTOnHelix.proj $XHARNESS_OUT/AOTBuild.binlog <_AOTBuildCommand Condition="'$(ContinuousIntegrationBuild)' != 'true'">$(_AOTBuildCommand) /p:RuntimeSrcDir=$(RepoRoot) /p:RuntimeConfig=$(Configuration) - <_AOTBuildCommand>$(_AOTBuildCommand) /p:RunAOTCompilation=$(RunAOTCompilation) /p:EmccLinkOptimizationFlag='-Oz -Wl%252C-O0 -Wl%252C-lto-O0' + <_AOTBuildCommand>$(_AOTBuildCommand) /p:RunAOTCompilation=$(RunAOTCompilation) <_AOTBuildCommand>$(_AOTBuildCommand) && cd wasm_build/AppBundle $(_AOTBuildCommand) @@ -87,9 +87,15 @@ AssemblyFile="$(WasmBuildTasksAssemblyPath)" /> + + true + -O2 + + <_MainAssemblyPath Condition="'%(WasmAssembliesToBundle.FileName)' == $(AssemblyName) and '%(WasmAssembliesToBundle.Extension)' == '.dll'">%(WasmAssembliesToBundle.Identity) $([System.IO.Path]::ChangeExtension($(_MainAssemblyPath), '.runtimeconfig.json')) + -Oz -Wl%252C-O0 -Wl%252C-lto-O0 @@ -111,15 +117,20 @@ - <_WasmPropertyNames Include="InvariantGlobalization" /> <_WasmPropertyNames Include="AOTMode" /> - <_WasmPropertyNames Include="WasmDebugLevel" /> + <_WasmPropertyNames Include="AssemblyName" /> + <_WasmPropertyNames Include="DisableParallelAot" /> + <_WasmPropertyNames Include="DisableParallelEmccCompile" /> + <_WasmPropertyNames Include="EmccCompileOptimizationFlag" /> + <_WasmPropertyNames Include="EmccLinkOptimizationFlag" /> + <_WasmPropertyNames Include="IncludeSatelliteAssembliesInVFS" /> + <_WasmPropertyNames Include="InvariantGlobalization" /> <_WasmPropertyNames Include="WasmBuildNative" /> - <_WasmPropertyNames Include="_WasmDevel" /> - <_WasmPropertyNames Include="WasmLinkIcalls" /> + <_WasmPropertyNames Include="WasmDebugLevel" /> <_WasmPropertyNames Include="WasmDedup" /> - <_WasmPropertyNames Include="IncludeSatelliteAssembliesInVFS" /> - <_WasmPropertyNames Include="AssemblyName" /> + <_WasmPropertyNames Include="WasmLinkIcalls" /> + <_WasmPropertyNames Include="WasmNativeStrip" /> + <_WasmPropertyNames Include="_WasmDevel" /> <_WasmPropertiesToPass Include="$(%(_WasmPropertyNames.Identity))" From 9a6d859ed82b40bf1890550e53032f34758be267 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:39:38 -0400 Subject: [PATCH 5/8] [wasm] Enable source generators tests that were failing due to OOM - `Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests` - `Microsoft.Extensions.Logging.Generators.Roslyn4.0.Tests` - `System.Text.Json.SourceGeneration.Roslyn3.11.Unit.Tests` - `System.Text.Json.SourceGeneration.Roslyn4.0.Unit.Tests` - `System.Text.RegularExpressions.Generators.Tests` Some individual ones are disabled due to https://github.com/dotnet/runtime/issues/58226, and https://github.com/dotnet/runtime/issues/60899 . Fixes https://github.com/dotnet/runtime/issues/51961 . --- ...Extensions.Logging.Generators.Roslyn3.11.Tests.csproj | 1 + ....Extensions.Logging.Generators.Roslyn4.0.Tests.csproj | 3 +++ .../JsonSourceGeneratorDiagnosticsTests.cs | 4 ++++ .../JsonSourceGeneratorTests.cs | 1 + ...xt.Json.SourceGeneration.Roslyn3.11.Unit.Tests.csproj | 1 + ...ext.Json.SourceGeneration.Roslyn4.0.Unit.Tests.csproj | 1 + .../TypeWrapperTests.cs | 2 ++ .../tests/RegexCultureTests.cs | 1 + ...ystem.Text.RegularExpressions.Generators.Tests.csproj | 1 + src/libraries/tests.proj | 9 --------- 10 files changed, 15 insertions(+), 9 deletions(-) diff --git a/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests.csproj b/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests.csproj index bb4f032a25357..e3ac6c40c52f2 100644 --- a/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests.csproj +++ b/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn3.11.Tests.csproj @@ -2,6 +2,7 @@ $(MicrosoftCodeAnalysisCSharpWorkspacesVersion_3_11) + true diff --git a/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn4.0.Tests.csproj b/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn4.0.Tests.csproj index 1c826c0b30a3f..074da27c19e02 100644 --- a/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn4.0.Tests.csproj +++ b/src/libraries/Microsoft.Extensions.Logging.Abstractions/tests/Microsoft.Extensions.Logging.Generators.Tests/Microsoft.Extensions.Logging.Generators.Roslyn4.0.Tests.csproj @@ -3,6 +3,9 @@ $(MicrosoftCodeAnalysisCSharpWorkspacesVersion) $(DefineConstants);ROSLYN4_0_OR_GREATER + true + -O1 + false diff --git a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorDiagnosticsTests.cs b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorDiagnosticsTests.cs index 470c83408c2c1..a25196a34fe07 100644 --- a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorDiagnosticsTests.cs +++ b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorDiagnosticsTests.cs @@ -125,6 +125,7 @@ public class IndexViewModel } [Fact] + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public void NameClashSourceGeneration() { // Without resolution. @@ -149,6 +150,7 @@ public void NameClashSourceGeneration() } [Fact] + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public void ProgramsThatDontUseGeneratorCompile() { // No STJ usage. @@ -192,6 +194,7 @@ public static void Main() } [Fact] + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public void WarnOnClassesWithInitOnlyProperties() { Compilation compilation = CompilationHelper.CreateCompilationWithInitOnlyProperties(); @@ -206,6 +209,7 @@ public void WarnOnClassesWithInitOnlyProperties() } [Fact] + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public void WarnOnClassesWithInaccessibleJsonIncludeProperties() { Compilation compilation = CompilationHelper.CreateCompilationWithInaccessibleJsonIncludeProperties(); diff --git a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorTests.cs b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorTests.cs index 105dc91af17b3..08519e16fde39 100644 --- a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorTests.cs +++ b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/JsonSourceGeneratorTests.cs @@ -11,6 +11,7 @@ namespace System.Text.Json.SourceGeneration.UnitTests { + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public class GeneratorTests { [Fact] diff --git a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn3.11.Unit.Tests.csproj b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn3.11.Unit.Tests.csproj index 5a99fa881f97f..af08a0ae33acc 100644 --- a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn3.11.Unit.Tests.csproj +++ b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn3.11.Unit.Tests.csproj @@ -1,6 +1,7 @@ $(MicrosoftCodeAnalysisCSharpWorkspacesVersion_3_11) + true diff --git a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn4.0.Unit.Tests.csproj b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn4.0.Unit.Tests.csproj index 073d9ca3126c7..709f667c267a2 100644 --- a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn4.0.Unit.Tests.csproj +++ b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/System.Text.Json.SourceGeneration.Roslyn4.0.Unit.Tests.csproj @@ -2,6 +2,7 @@ $(MicrosoftCodeAnalysisCSharpWorkspacesVersion) $(DefineConstants);ROSLYN4_0_OR_GREATER + true diff --git a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/TypeWrapperTests.cs b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/TypeWrapperTests.cs index 24a86137bc175..ca6d9dce1bd1e 100644 --- a/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/TypeWrapperTests.cs +++ b/src/libraries/System.Text.Json/tests/System.Text.Json.SourceGeneration.Unit.Tests/TypeWrapperTests.cs @@ -14,6 +14,7 @@ namespace System.Text.Json.SourceGeneration.UnitTests public class TypeWrapperTests { [Fact] + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public void MetadataLoadFilePathHandle() { // Create a MetadataReference from new code. @@ -79,6 +80,7 @@ public void MySecondMethod() { } } [Fact] + [ActiveIssue("https://github.com/dotnet/runtime/issues/58226", TestPlatforms.Browser)] public void CanGetAttributes() { string source = @" diff --git a/src/libraries/System.Text.RegularExpressions/tests/RegexCultureTests.cs b/src/libraries/System.Text.RegularExpressions/tests/RegexCultureTests.cs index 9155e9a947688..b029347a1b142 100644 --- a/src/libraries/System.Text.RegularExpressions/tests/RegexCultureTests.cs +++ b/src/libraries/System.Text.RegularExpressions/tests/RegexCultureTests.cs @@ -325,6 +325,7 @@ public static IEnumerable Match_In_Different_Cultures_CriticalCases_Te public static IEnumerable Match_In_Different_Cultures_CriticalCases_TestData() => Match_In_Different_Cultures_CriticalCases_TestData_For(RegexOptions.None).Union(Match_In_Different_Cultures_CriticalCases_TestData_For(RegexOptions.Compiled)); + [ActiveIssue("https://github.com/dotnet/runtime/issues/60899", TestPlatforms.Browser)] [Theory] [MemberData(nameof(Match_In_Different_Cultures_TestData))] public void Match_In_Different_Cultures(string pattern, RegexOptions options, CultureInfo culture, string input, string match_expected) diff --git a/src/libraries/System.Text.RegularExpressions/tests/System.Text.RegularExpressions.Generators.Tests/System.Text.RegularExpressions.Generators.Tests.csproj b/src/libraries/System.Text.RegularExpressions/tests/System.Text.RegularExpressions.Generators.Tests/System.Text.RegularExpressions.Generators.Tests.csproj index 82dfb654c2e4a..17a78af31cc11 100644 --- a/src/libraries/System.Text.RegularExpressions/tests/System.Text.RegularExpressions.Generators.Tests/System.Text.RegularExpressions.Generators.Tests.csproj +++ b/src/libraries/System.Text.RegularExpressions/tests/System.Text.RegularExpressions.Generators.Tests/System.Text.RegularExpressions.Generators.Tests.csproj @@ -7,6 +7,7 @@ enable $(NoWarn);xUnit2008 + true diff --git a/src/libraries/tests.proj b/src/libraries/tests.proj index 913a97376183d..1b1e89c59af16 100644 --- a/src/libraries/tests.proj +++ b/src/libraries/tests.proj @@ -14,9 +14,6 @@ - - - @@ -223,12 +220,6 @@ - - - - - - From 4fb5bf466e0c0df2a9a2586fd89b0f4cd712f32f Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:42:55 -0400 Subject: [PATCH 6/8] [wasm] Browser.bench is not sent to helix, so don't archive it --- .../sample/wasm/browser-bench/Wasm.Browser.Bench.Sample.csproj | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mono/sample/wasm/browser-bench/Wasm.Browser.Bench.Sample.csproj b/src/mono/sample/wasm/browser-bench/Wasm.Browser.Bench.Sample.csproj index 01b36c00dc7ce..ff9f990c78caf 100644 --- a/src/mono/sample/wasm/browser-bench/Wasm.Browser.Bench.Sample.csproj +++ b/src/mono/sample/wasm/browser-bench/Wasm.Browser.Bench.Sample.csproj @@ -1,6 +1,7 @@ - true + + false runtime.js From 485d4810332cfdbfd26fd999ed8e3161dd286766 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Wed, 27 Oct 2021 17:43:46 -0400 Subject: [PATCH 7/8] [wasm] Bump sdk used for testing workloads to 6.0.100-rtm.21480.21 --- eng/Versions.props | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/Versions.props b/eng/Versions.props index 08c358f65f448..f9596a96d831f 100644 --- a/eng/Versions.props +++ b/eng/Versions.props @@ -163,7 +163,7 @@ 2.0.4 4.12.0 2.14.3 - 6.0.100-rc.2.21474.31 + 6.0.100-rtm.21480.21 1.1.1-beta1.21467.5 6.0.0-preview-20211019.1 From ddb1ffb3abc30d5ea5f72a64d30f2723a25420ad Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Fri, 29 Oct 2021 13:19:14 -0400 Subject: [PATCH 8/8] address review feedback --- eng/testing/WasmRunnerAOTTemplate.sh | 2 +- eng/testing/tests.wasm.targets | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/eng/testing/WasmRunnerAOTTemplate.sh b/eng/testing/WasmRunnerAOTTemplate.sh index 5eb2d76d0774b..6922b550551b8 100644 --- a/eng/testing/WasmRunnerAOTTemplate.sh +++ b/eng/testing/WasmRunnerAOTTemplate.sh @@ -25,7 +25,7 @@ elif [ -z "$XHARNESS_COMMAND" ]; then XHARNESS_COMMAND="test" fi -function BuildAOT() +function _buildAOTFunc() { local projectFile=$1 local binLog=$2 diff --git a/eng/testing/tests.wasm.targets b/eng/testing/tests.wasm.targets index 1554237bf38ff..00aa7b19f1701 100644 --- a/eng/testing/tests.wasm.targets +++ b/eng/testing/tests.wasm.targets @@ -38,7 +38,7 @@ - <_AOTBuildCommand>BuildAOT publish/ProxyProjectForAOTOnHelix.proj $XHARNESS_OUT/AOTBuild.binlog + <_AOTBuildCommand>_buildAOTFunc publish/ProxyProjectForAOTOnHelix.proj $XHARNESS_OUT/AOTBuild.binlog <_AOTBuildCommand Condition="'$(ContinuousIntegrationBuild)' != 'true'">$(_AOTBuildCommand) /p:RuntimeSrcDir=$(RepoRoot) /p:RuntimeConfig=$(Configuration)