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 #53

Merged
merged 1 commit into from Oct 13, 2015

Conversation

Projects
None yet
1 participant
@jtimberman
Member

jtimberman commented Oct 13, 2015

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.

Relax version constraints for dependencies
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

@jtimberman jtimberman merged commit bb09569 into master Oct 13, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jtimberman jtimberman deleted the jtimberman/relax-constraint branch Oct 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment