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

Forge module version is ignored and support 1.x type versions #60

Merged
merged 1 commit into from
Dec 10, 2013

Conversation

carlossg
Copy link
Collaborator

Added the cucumber tests for both cases, tested on 3.0.1 and 2.7.14

Supersedes #56 and #45

@cprice404
Copy link

Thanks for submitting this @carlossg ! We've had this come up as an issue on at least two different puppet modules recently.

@craigtracey
Copy link
Contributor

Yes, thanks @carlossg. We are also encountering this.

@dgoodlad
Copy link

+1; librarian-puppet was unusable with recent modules without this patch, but works great with it applied directly to 0.9.7.

:shipit:

@dgoodlad
Copy link

dgoodlad commented Dec 4, 2012

@rodjek What's your opinion of these changes?

It seems to me like the disconnect between the versioning style being promoted by Puppet Labs and that of Rubygems (and therefore of librarian) is growing. The 0.x style, and now the puppetlabs-apache module using a 0.5.0-rc1 style instead of 0.5.0.rc1 is just going to continue to bite users of puppet-librarian.

I did a quick hack, though it's not tied in very nicely, on top of this PR to start building some generic mappings back and forth between puppet/gem style version specs: dgoodlad@996eb9a . It feels like it might be the right direction to head...

Thoughts?

@patcon
Copy link
Collaborator

patcon commented Mar 11, 2013

👍

@carlossg
Copy link
Collaborator Author

--version fixed in 9a291d8

@patcon
Copy link
Collaborator

patcon commented Nov 26, 2013

❤️

Would it be too much trouble to work through PR's @carlossg?

@carlossg
Copy link
Collaborator Author

@patcon yeah, trying to rebase and match the multiple commits to the multiple issues

carlossg added a commit that referenced this pull request Dec 10, 2013
1.x versions should translate to ~>1.0 gem requirement
@carlossg carlossg merged commit f7aff5c into rodjek:master Dec 10, 2013
@carlossg carlossg deleted the forge-versioning branch December 10, 2013 20:25
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.

5 participants