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.

Sign up for GitHub

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

Jump to bottom

Framework dependencies handled too strict #1206

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

Framework dependencies handled too strict #1206

Stift opened this issue Nov 10, 2015 · 4 comments
Labels
bug

Comments

@Stift
Copy link
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
Copy link
Member

forki commented Nov 10, 2015

Could you please create a minimal deps file?

@Stift
Copy link
Contributor Author

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 as completed in c8789b5 Nov 10, 2015
@forki
Copy link
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
Copy link
Contributor Author

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
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@forki @Stift