Git Product home page Git Product logo

Comments (13)

KevinMarquette avatar KevinMarquette commented on August 16, 2024 2

The core issue driving this issue in nuget here: NuGet/Home#3050

Either drop to a 3 part version number or place anything except 0 in the 4th part. This will mitigate a lot of frustration for people trying to use an internal PSRepository.

from dscresources.

ThomasNieto avatar ThomasNieto commented on August 16, 2024 1

@johlju PowerShellGet does a version comparison so 3.1.0 is newer than 3.0.0.0.

C:\> [version]'3.1.0' -gt [version]'3.0.0.0'
True

from dscresources.

johlju avatar johlju commented on August 16, 2024 1

The modules are released using the version in the module manifest which is using 4-part version. It’s a manual process where most likely Publish-Module is used.

In the first module automatically released in DSC Community https://github.com/dsccommunity/GPRegistryPolicyDsc we switch to using 3-part version number.

from dscresources.

johlju avatar johlju commented on August 16, 2024 1

All modules transferred to DSC Community can now be converted using the new CI pipeline which uses semantic versioning. Community has already converted some.
https://dsccommunity.org/blog/convert-a-module-for-continuous-delivery/

from dscresources.

gaelcolas avatar gaelcolas commented on August 16, 2024

Hi @tnieto88,
Thanks for the feedback.

I'm keen to use SemVer for many reasons, as long as we keep the NuGet compliant format for pre-releases.
Ideally, in my opinion, we'd use something like gitversion to consistently version our releases.
The problem here is that it's quite a bit of a change on our current release process, so it'll take some time to transition.
We're already looking at ways to make the release process a bit more transparent so that the community can contribute.
Stay tuned! ;)

from dscresources.

ThomasNieto avatar ThomasNieto commented on August 16, 2024

@gaelcolas My main concern is that DSC resources are using four digit version with a 0 in forth position. NuGet will normalize that version that to a three digit version number causing the module manifest and package version not to match. For more information about NuGet version normalization: https://docs.microsoft.com/en-us/nuget/reference/package-versioning#normalized-version-numbers

To resolve this issue the DSC modules will need to use a three digit versions that will not get normalized. From what I've seen in the last year of releases DSC resources never uses the fourth position so it shouldn't be an issue migrating from four digit to three digit version.

from dscresources.

PlagueHO avatar PlagueHO commented on August 16, 2024

I reckon we're going to need to get a fix for this sooner rather than later and drop the trailing number if it is a 0 when publishing/deploying as @KevinMarquette says. This will require modifications to the publish scripts that @kwirkykat wrote, but I don't see that is a major problem - should be a simple change. But it will mean the resources drop the last digit in the PS Gallery (because they're always a zero).

@johlju , @gaelcolas - do you see a problem with us doing this before the next release? Would need to run it past Katie I'd think.

from dscresources.

johlju avatar johlju commented on August 16, 2024

I have no issue with changing, but what happens when users run Update-Module? Will it see it as a new version if existing modules are present? Will it see 3.1.0 as higher than 3.0.0.0? πŸ€”

from dscresources.

StingyJack avatar StingyJack commented on August 16, 2024

NuGet will normalize versions to be semver, in DSC case removing the trailing zero (2.3.0.0 to 2.3.0).

@tnieto88 - it only removes the trailing zeros and some leading zeros. So you can have a 2.3.0.1, but it will change your 2.03.0 or 2.3.0.0 to 2.3.0.

It would be great if this group does not make the same mistakes that the NuGet team has made in this regard. The intention for version normalization in nuget was to make it easier for nuget.org to sanely deal with package versions on such a massive scale. The mistake was that it should have only been made a requirement and filter for packages published to that repository and others that elect to impose that restriction. It should not have been forced on every non-public nuget repo by changing a value the program is expected to take verbatim.

If you like irony, in the case of NuGet, they are trying to coerce a flavor of SemVer and knowingly introduced a breaking change to do it, but did not update the major version number (which SemVer dictates should happen).

from dscresources.

KevinMarquette avatar KevinMarquette commented on August 16, 2024

How is the DSC Resource Kit published so that the 4 part version is showing up correctly in PSGallery? The core issue here is that Publish-Module is using NuGet and NuGet is packaging them in a different way.

from dscresources.

KevinMarquette avatar KevinMarquette commented on August 16, 2024

I was only asking about publishing because Publish-Module is handing the work off to nuget. The publishing process must be using a down level version of nuget to preserve the 4 part version number for the gallery.

from dscresources.

johlju avatar johlju commented on August 16, 2024

@KevinMarquette Ah, now I understand what you meant. Meaning we could have an issue deploying 4-part in the future if Publish-Module switches to a newer version of nuget? πŸ€”

from dscresources.

KevinMarquette avatar KevinMarquette commented on August 16, 2024

More specifically, if the system running Publish-Module updates NuGet then you will also have this issue. Publish-Module doesn't pin to specific version and that is why people are running into this issue. If Publish-Module would handle it, then nobody would have the issue.

from dscresources.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.