Skip to content

Commit 6ab84e0

Browse files
authored
Move System.Data.SqlClient version from csproj to dependencies.props (#786)
1 parent 0f114f9 commit 6ab84e0

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

build/dependencies.props

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,14 @@
33
<AspNetCoreVersion>2.0.0-preview1-*</AspNetCoreVersion>
44
<AspNetIntegrationTestingVersion>0.4.0-*</AspNetIntegrationTestingVersion>
55
<AspNetCoreModuleVersion>1.0.0-*</AspNetCoreModuleVersion>
6+
7+
<!--
8+
* Use 4.4.0-* instead of $(CoreFxVersion) to address "SqlClient fails with netcoreapp2.0 on Win7/Server2008"
9+
* https://github.com/dotnet/corefx/issues/18406
10+
* Revert if and when $(CoreFxVersion) is upgraded to 4.4.0-*
11+
-->
12+
<SqlClientVersion>4.4.0-*</SqlClientVersion>
13+
614
<CoreFxVersion>4.3.0</CoreFxVersion>
715
<NETStandardImplicitPackageVersion>$(BundledNETStandardPackageVersion)</NETStandardImplicitPackageVersion>
816
<TestSdkVersion>15.0.0</TestSdkVersion>

test/MusicStore.E2ETests/MusicStore.E2ETests.csproj

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,11 @@
3030
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="$(TestSdkVersion)" />
3131

3232
<!--
33-
* Use 4.4.0-* instead of $(CoreFxVersion) to address "SqlClient fails with netcoreapp2.0 on Win7/Server2008"
33+
* Use $(SqlClientVersion) instead of $(CoreFxVersion) to address "SqlClient fails with netcoreapp2.0 on Win7/Server2008"
3434
* https://github.com/dotnet/corefx/issues/18406
35-
* Revert if and when $(CoreFxVersion) is upgraded to 4.4.0-*
35+
* Revert if and when $(CoreFxVersion) is upgraded to match $(SqlClientVersion)
3636
-->
37-
<PackageReference Include="System.Data.SqlClient" Version="4.4.0-*" />
37+
<PackageReference Include="System.Data.SqlClient" Version="$(SqlClientVersion)" />
3838

3939
<PackageReference Include="System.Xml.XmlDocument" Version="$(CoreFxVersion)" />
4040
<PackageReference Include="xunit.runner.visualstudio" Version="$(XunitVersion)" />

0 commit comments

Comments
 (0)