Skip to content

[release/5.0-preview3] Fix System.Data.SqlClient baseline (#34352) - #34397

Merged
danmoseley merged 1 commit into
dotnet:release/5.0-preview3from
mmitche:cherry-pick-sqlclient-fix
Apr 1, 2020
Merged

[release/5.0-preview3] Fix System.Data.SqlClient baseline (#34352)#34397
danmoseley merged 1 commit into
dotnet:release/5.0-preview3from
mmitche:cherry-pick-sqlclient-fix

Conversation

@mmitche

@mmitche mmitche commented Apr 1, 2020

Copy link
Copy Markdown
Member

Fix the package baseline for Microsoft.Windows.Compatibility so that the System.Data.SqlClient dependency is 4.8.1

Customer Impact

In preview1 and 2, Microsoft.Windows.Compatibility is dependent on System.Data.SqlClient 5.0.0-preview.X. System.Data.SqlClient is currently at 4.8.1, so this makes the package unusable.

Regression?

Yes,

Risk

Low, fix in master has been verified.

* Fix System.Data.SqlClient version

Fixes dotnet#34351

* Update Microsoft.Windows.Compatibility.pkgproj
@mmitche
mmitche requested a review from ericstj April 1, 2020 14:32
@danmoseley danmoseley added the Servicing-consider Issue for next servicing release review label Apr 1, 2020
@danmoseley

Copy link
Copy Markdown
Contributor

Supportive. Is there a checklist we need to tweak to avoid this next time?

@ViktorHofer

Copy link
Copy Markdown
Member

cc @jkotas

@jkotas jkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@danmoseley danmoseley added this to the 5.0 Preview 3 milestone Apr 1, 2020
@danmoseley
danmoseley merged commit 73da9bb into dotnet:release/5.0-preview3 Apr 1, 2020
@danmoseley

Copy link
Copy Markdown
Contributor

was approved

@danmoseley danmoseley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Apr 1, 2020
@ericstj

ericstj commented Apr 1, 2020

Copy link
Copy Markdown
Member

Is there a checklist we need to tweak to avoid this next time?

A checklist for deleting a library from the repo? Not sure we have such a thing or if it happens often enough.

I do think we should test the compat pack during package testing. @safern can you file an issue for that? I suspect it's not being tested because it's not validated for support.

@ericstj

ericstj commented Apr 1, 2020

Copy link
Copy Markdown
Member

Nevermind, we have that issue: #27503

@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants