From 0495211bd4a2e3b766f1d5bd7714ddb0f8534252 Mon Sep 17 00:00:00 2001 From: Kyungwoo Lee Date: Fri, 4 Mar 2016 12:35:13 -0800 Subject: [PATCH] Fix for not using ilasm from the build binary Using ilasm from the build is a desire to test ilasm tool -- in fact we've now had ildasm-ilasm round-trip tests, so this is not critical either. But when building cross-target like ARM64, running such native ilasm in the different host (x64) is not an option. Given the current test architect where tests are all populated in Windows, I just fall back to use one from VS tool same as other tools like CSC. Ideally, we should download such host tools based on host target especially when we start populating tests on non-Window platforms. Currently two of tests are disabled due to the issue #3517. --- tests/issues.targets | 6 ++++++ tests/src/IL.targets | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/issues.targets b/tests/issues.targets index dd15f0c..276427a 100644 --- a/tests/issues.targets +++ b/tests/issues.targets @@ -232,6 +232,12 @@ 2441 + + 3517 + + + 3517 + 2441 diff --git a/tests/src/IL.targets b/tests/src/IL.targets index e2be217..36cff53 100644 --- a/tests/src/IL.targets +++ b/tests/src/IL.targets @@ -22,7 +22,7 @@ <_IlasmSwitches Condition="'$(Optimize)' == 'True'">$(_IlasmSwitches) -OPTIMIZE - + -- 2.7.4