Move the PreBuild dependency property later in the file so targets can't override it (#10429)

* Move the PreBuild dependency property later in the file so targets can't override it

* [csproj] Update project files

9 files changed, 54 insertions, 36 deletions

diff --git a/mcs/nunit24/ClientUtilities/util/nunit.util.csproj b/mcs/nunit24/ClientUtilities/util/nunit.util.csproj
index b16012f9209..ba33aa61238 100644
--- a/mcs/nunit24/ClientUtilities/util/nunit.util.csproj
+++ b/mcs/nunit24/ClientUtilities/util/nunit.util.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -126,4 +122,10 @@
     </EmbeddedResource>

   </ItemGroup>

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/ConsoleRunner/nunit-console-exe/nunit-console.csproj b/mcs/nunit24/ConsoleRunner/nunit-console-exe/nunit-console.csproj
index 00bdd57f358..0695e61c8da 100644
--- a/mcs/nunit24/ConsoleRunner/nunit-console-exe/nunit-console.csproj
+++ b/mcs/nunit24/ConsoleRunner/nunit-console-exe/nunit-console.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -63,4 +59,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/ConsoleRunner/nunit-console/nunit-console-runner.csproj b/mcs/nunit24/ConsoleRunner/nunit-console/nunit-console-runner.csproj
index f4253f04e17..a2e0e2d083d 100644
--- a/mcs/nunit24/ConsoleRunner/nunit-console/nunit-console-runner.csproj
+++ b/mcs/nunit24/ConsoleRunner/nunit-console/nunit-console-runner.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -68,4 +64,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/NUnitCore/core/nunit.core.csproj b/mcs/nunit24/NUnitCore/core/nunit.core.csproj
index bbf4246bf2d..dc8da1edcda 100644
--- a/mcs/nunit24/NUnitCore/core/nunit.core.csproj
+++ b/mcs/nunit24/NUnitCore/core/nunit.core.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -126,4 +122,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/NUnitCore/interfaces/nunit.core.interfaces.csproj b/mcs/nunit24/NUnitCore/interfaces/nunit.core.interfaces.csproj
index 96543c59a73..9d104f30122 100644
--- a/mcs/nunit24/NUnitCore/interfaces/nunit.core.interfaces.csproj
+++ b/mcs/nunit24/NUnitCore/interfaces/nunit.core.interfaces.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -102,4 +98,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/NUnitExtensions/core/nunit.core.extensions.csproj b/mcs/nunit24/NUnitExtensions/core/nunit.core.extensions.csproj
index f7e6f087cc1..eccb1b6d7a7 100644
--- a/mcs/nunit24/NUnitExtensions/core/nunit.core.extensions.csproj
+++ b/mcs/nunit24/NUnitExtensions/core/nunit.core.extensions.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -71,4 +67,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/NUnitExtensions/framework/nunit.framework.extensions.csproj b/mcs/nunit24/NUnitExtensions/framework/nunit.framework.extensions.csproj
index 7a1892cf1d9..d68a0b7922a 100644
--- a/mcs/nunit24/NUnitExtensions/framework/nunit.framework.extensions.csproj
+++ b/mcs/nunit24/NUnitExtensions/framework/nunit.framework.extensions.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -68,4 +64,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/NUnitFramework/framework/NUnit.Framework.csproj b/mcs/nunit24/NUnitFramework/framework/NUnit.Framework.csproj
index 1ba0f7f1338..d5b6f4c8ea2 100644
--- a/mcs/nunit24/NUnitFramework/framework/NUnit.Framework.csproj
+++ b/mcs/nunit24/NUnitFramework/framework/NUnit.Framework.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -113,4 +109,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file
diff --git a/mcs/nunit24/NUnitMocks/mocks/nunit.mocks.csproj b/mcs/nunit24/NUnitMocks/mocks/nunit.mocks.csproj
index 49a83941450..ae0c86656d1 100644
--- a/mcs/nunit24/NUnitMocks/mocks/nunit.mocks.csproj
+++ b/mcs/nunit24/NUnitMocks/mocks/nunit.mocks.csproj
@@ -22,10 +22,6 @@
     Microsoft.NETFramework.props will force a dependency on the assembly System.Core. This

     is a problem to compile the Mono mscorlib.dll -->

     <AddAdditionalExplicitAssemblyReferences>false</AddAdditionalExplicitAssemblyReferences>

-    <!-- Force the pre-build event to run after references have been resolved. The default

-      behavior is to run them before resolving references, which can cause things like

-      culevel.exe to be used before they have been built. -->

-    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

   </PropertyGroup>

   <PropertyGroup Condition=" '$(Platform)' == 'net_4_x' ">

     <OutputPath>./../../../class/lib/net_4_x-$(HostPlatform)</OutputPath>

@@ -72,4 +68,10 @@
   </ItemGroup>

   <!-- @ALL_REFERENCES@ -->

   <!-- @ALL_RESOURCES@ -->

+  <PropertyGroup>

+    <!-- Force the pre-build event to run after references have been resolved. The default

+      behavior is to run them before resolving references, which can cause things like

+      culevel.exe to be used before they have been built. -->

+    <PreBuildEventDependsOn>ResolveReferences</PreBuildEventDependsOn>

+  </PropertyGroup>

 </Project>
\ No newline at end of file