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

Paket v4 fails update cmd (infinite dependency loop ?) #2225

Closed
MrLuje opened this issue Apr 10, 2017 · 4 comments
Closed

Paket v4 fails update cmd (infinite dependency loop ?) #2225

MrLuje opened this issue Apr 10, 2017 · 4 comments

Comments

@MrLuje
Copy link
Contributor

MrLuje commented Apr 10, 2017

Description

I have a solution where paket (4.3.0 at this time) fails the update command, it never ends
whereas with paket 3.36, it finished in 4s

Repro steps

Not ATM, reproduced with a private repo, I will try to provide something if I can pinpoint the issue

Expected behavior

Paket update should finish in a reasonable amount time

Actual behavior

I stopped the update after 30 min
I attached a filtered verbose output for both v3 & v4 paket.exe

v4_res.txt
v3_res.txt

You can see that v4 seems lost in the dependency graph, not sure if caused by a circular dependency but the thing is it was working fine with v3

Known workarounds

Stick with v3 for this solution 😢

@forki
Copy link
Member

forki commented Apr 10, 2017 via email

@MrLuje
Copy link
Contributor Author

MrLuje commented Apr 10, 2017

You can check https://github.com/MrLuje/Paket-v4-update-issue

I was not able to properly reproduce it so I integrated the same dependency graph we currently have on our private repo (as a local repository in the sln)

We found the cause : MassTransit & StackExchange.Redis have some incompatible versions (we can see this in verbose output) but it should be able to stop crawling the packages 😄

@forki
Copy link
Member

forki commented Apr 10, 2017

/cc @cloudRoutine. I think we should debug this one

@MrLuje
Copy link
Contributor Author

MrLuje commented May 11, 2017

In the end, duplicate of #2289 and also fixed by PR #2299
Great job, can't wait to see this officially released !

@MrLuje MrLuje closed this as completed May 11, 2017
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

2 participants