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

Tranistive package shrinks usage of frameworks #1215

Closed
Stift opened this issue Nov 12, 2015 · 7 comments
Closed

Tranistive package shrinks usage of frameworks #1215

Stift opened this issue Nov 12, 2015 · 7 comments
Labels

Comments

@Stift
Copy link
Contributor

Stift commented Nov 12, 2015

I use the following nuget package Microsoft.Bcl.Async in a Silverlight 5 project and this was fine so far. But now I added in my paket.dependencies file the nuget package OwinMiddlewares.Log4net. Because of that paket finds out that Microsoft.Bcl.Async is only needed in for framework >= net40, which is fine. But it now put this constraint as well in the paket.lock file on the pinned version and now in the csproj file no version is added for Silverlight 5.0. And this is not fine.

I have created a small sample. This contains 2 paket.dependency files and one project. The first dep file works fine, the second one removes the needed project reference
https://www.dropbox.com/s/puz3ustm5etr9wm/paket.zip?dl=1

@forki
Copy link
Member

forki commented Nov 12, 2015

since I worked on similar issue a couple of minutes ago - could you please verify that this still happens with latest? thx

@forki
Copy link
Member

forki commented Nov 12, 2015

yep seems still to be the case. confirmed => bug

@forki forki added the bug label Nov 12, 2015
@Stift
Copy link
Contributor Author

Stift commented Nov 12, 2015

Yep. Just tested it with 2.24.5 and still there.

@forki
Copy link
Member

forki commented Nov 12, 2015

I'm already deplyoing a fix ;-)

@forki
Copy link
Member

forki commented Nov 12, 2015

unfortunately all the integration test make this process much slower than a couple of weeks ago ;-)

@forki forki closed this as completed in 6185b89 Nov 12, 2015
@Stift
Copy link
Contributor Author

Stift commented Nov 12, 2015

2.24.6 works - very nice.
Will test it tomorrow in my office, but in my sample it looks promising.

@Stift
Copy link
Contributor Author

Stift commented Nov 13, 2015

Aaaand now its working completely. Thx again!

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

No branches or pull requests

2 participants