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

Support chef 12.8.1 64bits #21

Merged
merged 1 commit into from Apr 1, 2016

Conversation

Projects
None yet
2 participants
@kamaradclimber
Member

kamaradclimber commented Mar 31, 2016

Due to chef being in 32 bits, sysnative path was used to access
wuauclt.exe.
Now that chef 12.8.1 has a 64bits version, we have to adapt by using
powershell_script resource that make calls to System32 path works
depending on the node architecture (instead of the ruby architecture).

Support chef 12.8.1 64bits
Due to chef being in 32 bits, sysnative path was used to access
wuauclt.exe.
Now that chef 12.8.1 has a 64bits version, we have to adapt by using
powershell_script resource that make calls to System32 path works
depending on the node architecture (instead of the ruby architecture).
@kamaradclimber

This comment has been minimized.

Show comment
Hide comment
@kamaradclimber
Member

kamaradclimber commented Mar 31, 2016

@Annih

This comment has been minimized.

Show comment
Hide comment
@Annih

Annih Mar 31, 2016

Contributor

Same changes as #17 ... I feel bad for @jboeshart to have closed it now ^^'
Let's merge this, and finaly fix #16 even if I said we won't fix it

Contributor

Annih commented Mar 31, 2016

Same changes as #17 ... I feel bad for @jboeshart to have closed it now ^^'
Let's merge this, and finaly fix #16 even if I said we won't fix it

@kamaradclimber kamaradclimber merged commit c6adeb5 into master Apr 1, 2016

2 checks passed

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

@Annih Annih deleted the architecture branch Apr 1, 2016

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