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

Bug/13 download terraform loop #20

Merged
merged 3 commits into from
Oct 15, 2021

Conversation

cedrox
Copy link
Contributor

@cedrox cedrox commented Oct 14, 2021

related to https://github.com/magodo/aztfy/issues/13

tfinstall.ExecPathFinder need to have ExactPath.
On Windows it download terraform each time because the string compare is done on terraform and not terraform.exe

Cedric Folliot and others added 3 commits October 14, 2021 19:10
@magodo
Copy link
Collaborator

magodo commented Oct 15, 2021

@cedrox Thank you for looking into this and finding the root cause 👍. I've made another commit on top of yours, which both doing some cleanup to remove the debug related stuff (to make the code cleaner and the PR more focus on the issue), and used another way to differentiate between windows and non-windows by using Go build constraint (see: https://pkg.go.dev/cmd/go#hdr-Build_constraints for details).

@cedrox
Copy link
Contributor Author

cedrox commented Oct 15, 2021

Much cleaner !
Some guidance about env var and how to use it should be interesting.
Thanks

@magodo magodo merged commit 50663b4 into Azure:main Oct 15, 2021
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

2 participants