Skip to content

Commit 82772af

Browse files
committed
Remove OS constants from manual tests project
1 parent b1ed25a commit 82772af

1 file changed

Lines changed: 0 additions & 26 deletions

File tree

src/Microsoft.Data.SqlClient/tests/ManualTests/Microsoft.Data.SqlClient.ManualTests.csproj

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -8,32 +8,6 @@
88
<!-- This is only required today because of the separate test sets -->
99
<EnableDefaultCompileItems>false</EnableDefaultCompileItems>
1010
</PropertyGroup>
11-
12-
<!-- OS Constants ==================================================== -->
13-
<PropertyGroup>
14-
<!-- There are some test files in this project that only apply to a certain OS. However, -->
15-
<!-- because it isn't possible to run tests for the wrong OS (eg, built for Unix, run on -->
16-
<!-- Windows), this value shouldn't be modified except for aiding in development. As -->
17-
<!-- such, the preprocessor constants are set directly from $(OS). -->
18-
<TargetOs>$(OS)</TargetOs>
19-
20-
<!-- Uncomment one of the following lines to override the OS you are developing for. -->
21-
<!-- This is useful if your IDE doesn't show syntax highlighting for code that is not -->
22-
<!-- included via an #if. -->
23-
<!--<TargetOs>Unix</TargetOs>-->
24-
<!--<TargetOs>Windows_NT</TargetOs>-->
25-
26-
<NormalizedTargetOs>$(TargetOs.ToLower())</NormalizedTargetOs>
27-
28-
<!-- NOTE: These constants are prefixed with _ to keep them separate from .NET 5+ precompiler -->
29-
<!-- flags. Those only apply to OS-specific target frameworks, and would interfere here. -->
30-
<DefineConstants Condition="'$(NormalizedTargetOs)' == 'unix'">$(DefineConstants);_UNIX</DefineConstants>
31-
<DefineConstants Condition="'$(NormalizedTargetOs)' == 'windows_nt'">$(DefineConstants);_WINDOWS</DefineConstants>
32-
</PropertyGroup>
33-
<Target Name="ValidateOs" BeforeTargets="Build">
34-
<Error Condition="'$(TargetOS)' != '$(OS)'"
35-
Text="Cross-OS compilation of manual tests is not supported. Did you leave a development flag on?" />
36-
</Target>
3711

3812
<!-- Test Set ======================================================== -->
3913
<PropertyGroup>

0 commit comments

Comments
 (0)