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

Add the minimal fix to restore being able to compile on Illumos. #3024

Merged
merged 1 commit into from
May 9, 2017

Conversation

sean-
Copy link
Contributor

@sean- sean- commented May 9, 2017

This brings in the necessary fix to satisfy #2989 without dragging along
additional vendor updates reverted in #3019.

Pointy Hat: @sean- for not testing a release build w/ a clean GOPATH

This brings in the necessary fix to satisfy hashicorp#2989 without dragging along
additional vendor updates reverted in hashicorp#3019.
@jen20
Copy link
Contributor

jen20 commented May 9, 2017

An alternative way to fix this (though this was my fix that was merged into the Docker upstream) is to move the declaration of i out of the loop so it compiles with the defaults, though this is preferable.

@slackpad slackpad merged commit 3a8c6b8 into hashicorp:master May 9, 2017
@sean- sean- deleted the f-illumos-duex branch May 9, 2017 18:53
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