Skip to content

Conversation

@pghalliday
Copy link

For some reason I was consistently hitting the redirect limit with chef 11.10.4 even though there was only 1 redirect. This change gets rid of that 1 redirect.

@tombruijn
Copy link

Same problem here, just locked the Chef version until had time to investigate.
Don't know how many versions of chef ago it was introduced, worked on 11.6.x just fine. Might just be a Chef gem bug, but I hope this can be merged either way.

@brodock
Copy link

brodock commented Mar 22, 2014

It was not working on mine too... thanks for the fix

As a more permanent workaround i think we should use : http://docs.opscode.com/resource_git.html and clone/checkout the tag version directly

tombruijn added a commit to tombruijn/chef-chruby that referenced this pull request Mar 22, 2014
@tombruijn
Copy link

Made modifications to support git in pull request #20, but it has the downside of not being able to check the checksum of the installation. With git however we'd be rid of the new redirect limit and not have to worry about changes on GitHub's end.

@brodock
Copy link

brodock commented Mar 25, 2014

you don't need to checksum it because git already do it for you... if you really want to make sure you are getting what you think you are, just "checkout" by the commit hash

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.

3 participants