-
Notifications
You must be signed in to change notification settings - Fork 526
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
Convert-from-nuget modifies package versions X.Y.0 to X.Y #1591
Comments
Are you talking about the versions in the dependencies file? Can you manually add the 0 and check if install works then? |
When running verbose:
|
Actual error from artifactory:
Sounds actually more like an issue beween artifactory and public nuget to me. |
@cdrnet this is a known issue with many custom servers. Paket tries to work around that - seems we missed a spot. |
How can it work around that? |
@forki I have tried as specified in the workaround session to add the trailing 0, but the paket restore afterwards still tries to fetch 6.1 If you guys know a workaround, I could even use a custom build of Paket to migrate our remaining codebases. |
I'm already working on a fix. Hope to release in next half hour |
@forki you are more reactive than a premium gold support! |
@cdrnet what is the version that you see in the lock file? |
|
if you manually fix that, does it restore correctly? if you fix the versions in deps file and run update again, does it fix the lock file? |
I pushed a fix that would create the deps file with 6.1.0 instead of 6.1 |
With v2.58.11.0: A) No. It now does a lookup for 6.1.0 instead of 6.1, but that results in the same atom feed reply containing the same download URI (6.1.0) which still fails with 404. B) Yes, it updates the lock file to 6.1.0, but then still fails with 404. |
lol so I fixed completely wrong place ;-) |
I have just tested 2.58.12 and it does resolve the convert with Artifactory as a source. Thanks!!! |
So conversion is good but you still see wrong download url from
|
Yes, this seems to be a weird Artifactory issue, not directly related by the original issue and Paket (NPE actually crashes when trying to download that particular version). |
Description
When converting a codebase from NuGet to Paket using the command convert-with-paket, the process does modify package versions formed as X.Y.0 to X.Y
This causes no problem when the nuget source is nuget.org, but with Artifactory pro, the restore fails to find version X.Y, while X.Y.0 exists
Repro steps
Expected behavior
Paket shall pin EntityFramework to 6.1.0
Actual behavior
Paket will pin EntityFramework to version 6.1
Restore fails if the nuget source is Artifactory - I cannot share our Artifactory server for repro, it is only available from our Intranet
Known workarounds
I haven't found a workaround.
I have tried to run convert-from-nuget --no-install --no-auto-restore and then modify the version in paket.dependencies to 6.1.0, but the restore still fails, trying to look for 6.1
The text was updated successfully, but these errors were encountered: