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

Relax version constraints for dependencies #4

Merged
merged 1 commit into from Oct 13, 2015

Conversation

jtimberman
Copy link
Contributor

We need to use >= instead of ~> for version constraints because Chef
Policies use "dotted decimal identifier" versions that result in
integers that will never match the "normal" cookbook versions we use.

For example, the version constraint:

~> 3.2

is what we used. However, the dotted decimal version may be

30625036585770760.42672982950081083.162968991950205

The dotted decimal version is used in the "backwards compatibility
mode" for Chef Zero as it doesn't have the native Policy document API
(yet). This may result in a 412 precondition failed exception when
using the pessmistic version constraint.

We need to use >= instead of ~> for version constraints because Chef
Policies use "dotted decimal identifier" versions that result in
integers that will never match the "normal" cookbook versions we use.

For example, the version constraint:

~> 3.2

is what we used. However, the dotted decimal version may be

30625036585770760.42672982950081083.162968991950205

The dotted decimal version is used in the "backwards compatibility
mode" for Chef Zero as it doesn't have the native Policy document API
(yet). This may result in a 412 precondition failed exception when
using the pessmistic version constraint.
jtimberman added a commit that referenced this pull request Oct 13, 2015
Relax version constraints for dependencies
@jtimberman jtimberman merged commit 9d597da into master Oct 13, 2015
@jtimberman jtimberman deleted the jtimberman/relax-constraint branch October 13, 2015 21:29
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.

None yet

1 participant