Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reducing dependencies and references #1153

Closed
freica opened this issue Jul 7, 2021 · 7 comments
Closed

Reducing dependencies and references #1153

freica opened this issue Jul 7, 2021 · 7 comments

Comments

@freica
Copy link

freica commented Jul 7, 2021

I have 29 referenced assemblies on .NET 4.8 - using System.Data.SqlClient I have zero!
Is there a chance to reduce the references?

@ErikEJ
Copy link
Contributor

ErikEJ commented Jul 7, 2021

You are implicitly referencing everything in .NET Framework

@freica
Copy link
Author

freica commented Jul 7, 2021

Perhaps my statement was not specific enough.
Using Microsoft.Data.SqlClient I have to deploy 29 additional assemblies with my application, against 0 additional assemblies on using System.Data.SqlClient.

@ErikEJ
Copy link
Contributor

ErikEJ commented Jul 7, 2021

If you do not need the new features and performance improvements in M.D.S., you can keep using SDS.

Having package dependencies is how modern .net libraries work.

There are thoughts about splitting out the Azure dependencies.

@freica
Copy link
Author

freica commented Jul 7, 2021

We want to get ".NET5/6-ready" so we should move to M.D.S.

Having a small footprint is also a "modern feature"

Splitting out the Azure dependencies sounds good for me.

@ErikEJ
Copy link
Contributor

ErikEJ commented Jul 7, 2021

Possible duplicate of #1108 ?

@cheenamalhotra
Copy link
Member

Hi @freica

As @ErikEJ referenced, we're considering that change as part of #1108, but other dependencies would continue to stay as it is a standard .NET design requirement.

@freica
Copy link
Author

freica commented Jul 8, 2021

OK!

@freica freica closed this as completed Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants