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

Get package version from nuspec file #829

Merged
merged 1 commit into from
Jun 15, 2015
Merged

Conversation

tpetricek
Copy link
Member

I was using GetPackageVersion in FsLab. When we use Paket to get FsLab dependencies, it downloads the package Google.DataTable.Net.Wrapper and puts it in a file called Google.DataTable.Net.Wrapper.3.1.2.nupkg. However, the nuspec file says that the version is actually 3.1.2.0 and if you add the reference using NuGet, it places it in a directory with 3.1.2.0 in the name (and also includes the additional .0 in the nupkg file name).

I'm not sure if this is a bug in Paket (uses wrong file name) or if it is just NuGet giving us inconsistent information. I suspect the latter, so I'm sending a PR. But perhaps this is better handled in Paket.. not sure!

(Based on code I added to the FsLab build script: fslaborg/FsLab@2475145)

I was using `GetPackageVersion` in FsLab. When we use Paket to get FsLab dependencies, it
downloads the package `Google.DataTable.Net.Wrapper` and puts it in a file called
`Google.DataTable.Net.Wrapper.3.1.2.nupkg`. However, the `nuspec` file says that the version
is actually `3.1.2.0` and if you add the reference using NuGet, it places it in a directory
with `3.1.2.0` in the name (and also includes the additional `.0` in the  `nupkg` file name).

I'm not sure if this is a bug in Paket (uses wrong file name) or if it is just NuGet giving
us inconsistent information. I suspect the latter, so I'm sending a PR. But perhaps this is
better handled in Paket.. not sure!
@forki
Copy link
Member

forki commented Jun 15, 2015

Nuget changed the version normalize function along the way and is inconsistent.
the fix looks good

forki added a commit that referenced this pull request Jun 15, 2015
Get package version from nuspec file
@forki forki merged commit 6e37201 into fsprojects:master Jun 15, 2015
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

Successfully merging this pull request may close these issues.

2 participants