-
Notifications
You must be signed in to change notification settings - Fork 520
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
>= 0.0.0-prerelease
vs prerelease
#1088
Comments
Will look into this on Monday when I'm back from sailing
|
this is actually not supported. Why don't you ise
? |
when we use if we use |
But is that the correct nuspec syntax for prereleases?
|
http://docs.nuget.org/create/versioning#Specifying-Version-Ranges-in-.nuspec-Files: also paket update will resolve |
Ok so the bug is actually in the code that creates the new nuspec during
|
sounds reasonable to me. if it would apply the following transformations, i think it would mirror how paket update is resolving the packages:
|
What about alpha and beta versions? Paket is making a difference here
|
i dont think @devboy has tested the same with alpha & beta versions. unfortunately he is out of office this week. i can try and see if i can create a small testcase later. |
Nope haven't tested that! Greetings from Greece! |
Paket.Pack behaves the same when packing nupkgs that depend on alpha release. |
sorry I don't have a real repro sample at the moment, but I tried to fix it in the dark. could you please try with 2.4.8? |
thanks for looking into it so quickly. i only looked at it briefly so far but at first glance 2.4.8 does: |
mhm. I guess the last one is correct. Specifying a concrete version should always point to that version. |
Let's say I have:
Adding the nuget dependency:
nuget LibraryA >= 0.0.0-prerelease
will resolve to1.0.0
nuget LibraryA prerelease
will resolve to1.1.0-prerelease
When using Paket.Pack:
nuget LibraryA >= 0.0.0-prerelease
will add a dependency to">= 0.0.0-prerelease"
to the nuspecnuget LibraryA prerelease
will add a dependency to""
to the nuspecThe text was updated successfully, but these errors were encountered: