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

Framework dependencies handled too strict #1206

Closed
Stift opened this Issue Nov 10, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@Stift
Contributor

Stift commented Nov 10, 2015

My dependencies file containes log4net.ElasticSearch which has the transitive dependency to log4net for either net40 or net45. I narrow down my framework dependency to net40, net45, sl5. In that case my lock file does not show the dependency to log4net. I played around a little bit and removed the framework constraint of sl5 and the transitive dependency.
I have the feeling that paket understands the framework in an exclusive way which it shouldnt IMO.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 10, 2015

Member

Could you please create a minimal deps file?

Member

forki commented Nov 10, 2015

Could you please create a minimal deps file?

@Stift

This comment has been minimized.

Show comment
Hide comment
@Stift

Stift Nov 10, 2015

Contributor

Yep.
So this the not working dep file

framework: net40, net45, sl5
source https://nuget.org/api/v2

nuget log4net.ElasticSearch

and this dep file results in a correct lock file

framework: net40, net45
source https://nuget.org/api/v2

nuget log4net.ElasticSearch

I hope this helps!

Contributor

Stift commented Nov 10, 2015

Yep.
So this the not working dep file

framework: net40, net45, sl5
source https://nuget.org/api/v2

nuget log4net.ElasticSearch

and this dep file results in a correct lock file

framework: net40, net45
source https://nuget.org/api/v2

nuget log4net.ElasticSearch

I hope this helps!

@forki forki added the bug label Nov 10, 2015

@forki forki closed this in c8789b5 Nov 10, 2015

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 10, 2015

Member

latest release is generating:

FRAMEWORK: NET40, NET45, SL50
NUGET
  remote: https://nuget.org/api/v2
  specs:
    log4net (2.0.3) - framework: net40, net45
    log4net.ElasticSearch (2.2.0)
      log4net (>= 2.0.3.0) - framework: net40, net45
Member

forki commented Nov 10, 2015

latest release is generating:

FRAMEWORK: NET40, NET45, SL50
NUGET
  remote: https://nuget.org/api/v2
  specs:
    log4net (2.0.3) - framework: net40, net45
    log4net.ElasticSearch (2.2.0)
      log4net (>= 2.0.3.0) - framework: net40, net45
@Stift

This comment has been minimized.

Show comment
Hide comment
@Stift

Stift Nov 10, 2015

Contributor

Works as expected. Nice.

Contributor

Stift commented Nov 10, 2015

Works as expected. Nice.

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