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

.NET Standard 2.0 package #267

Closed
MihaMarkic opened this issue Jan 24, 2018 · 18 comments
Closed

.NET Standard 2.0 package #267

MihaMarkic opened this issue Jan 24, 2018 · 18 comments

Comments

@MihaMarkic
Copy link

Hi guys,

It'd be nice if you included a .NET Standard 2.0 package. Otherwise a lot of NuGet references is dragged into a Xamarin project.

@tmenier
Copy link
Owner

tmenier commented Feb 6, 2018

Makes sense to me. Will plan on getting this in the next release.

@tmenier tmenier added this to the Flurl.Http 2.1.2 milestone Feb 6, 2018
@kroniak
Copy link
Collaborator

kroniak commented Feb 8, 2018

Hi. You can use flurl.http 1.4 package in your 2.0 lib.

@MihaMarkic
Copy link
Author

@kroniak I know. But that's not the point.

@kroniak
Copy link
Collaborator

kroniak commented Feb 9, 2018 via email

@MihaMarkic
Copy link
Author

@kroniak As I wrote. When adding existing NuGet package to Xamarin, it would reference a ton of other NuGet packages. Which isn't the case if it is build for .NET Standard 2.

@tmenier
Copy link
Owner

tmenier commented Feb 16, 2018

@MihaMarkic Out of curiosity, is the "ton of other NuGet packages" just something developers see when they install Flurl or does it actually result in a more bloated compiled app that end-users install on their phone?

@tmenier
Copy link
Owner

tmenier commented Feb 16, 2018

@kroniak Everything builds fine by just adding netstandard2.0 to <TargetFrameworks> and making a tweak to one of the #if directives. However, would doing this "right" mean lots of changes to build/test scripts etc?

@kroniak
Copy link
Collaborator

kroniak commented Feb 17, 2018

@tmenier #273

@MihaMarkic
Copy link
Author

@tmenier TBH I do not know if there is a bloat added to the compiled app. It is bad enough (to me) that I get my nuget references polluted so I never went investigating what actually happens.

@tmenier
Copy link
Owner

tmenier commented Feb 17, 2018

@kroniak Thanks! I was already working on it myself and we ended up with the same results, which tells me I was probably doing it right. :)

Do you have an opinion on having package testers for both .NET Core 1.x and 2.x? I sometimes wonder if they're needed at all, but the way they share code, adding a new one requires almost no new code.

@kroniak
Copy link
Collaborator

kroniak commented Feb 17, 2018

@tmenier I think we can have only .netcore app 2.0

@tmenier
Copy link
Owner

tmenier commented Feb 18, 2018

@kroniak Is it strange that System.Reflection.TypeExtensions is required in netstandard1.3 but not 1.0 or 2.0?

https://github.com/tmenier/Flurl/blob/master/src/Flurl/Flurl.csproj#L28-L30

Could be right, I just wanted to make sure.

tmenier added a commit that referenced this issue Feb 18, 2018
@tmenier
Copy link
Owner

tmenier commented Feb 19, 2018

@MihaMarkic The latest prerelase contains a netstandard2.0 target. Please test it and let me know if it's what you're expecting:

https://www.nuget.org/packages/Flurl.Http/2.2.0-pre1

@kroniak
Copy link
Collaborator

kroniak commented Feb 19, 2018

@tmenier I think same. It is strange. But it does not need any package in std2.0 https://www.nuget.org/packages/System.Reflection.TypeExtensions/

@tmenier
Copy link
Owner

tmenier commented Feb 19, 2018

Well this isn't very nice:

dependencies

That's for the .NET Core 2.0 package tester. Fortunately I found the culprit - Json.NET 10.0.3 accounts for most of it. 11.x has a netstandard2.0 target with exactly zero dependencies. So needless to say there will be another prerelease.

@tmenier
Copy link
Owner

tmenier commented Feb 19, 2018

That's better...

dependencies

CodePages is necessary because of this.

@tmenier
Copy link
Owner

tmenier commented Feb 22, 2018

Full version is released.

@tmenier tmenier closed this as completed Feb 22, 2018
@tmenier tmenier removed the testing label Feb 22, 2018
@MihaMarkic
Copy link
Author

Thanks guys. It works fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants