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

Fix init function #294

Closed
wants to merge 3 commits into from
Closed

Fix init function #294

wants to merge 3 commits into from

Conversation

jukie
Copy link
Collaborator

@jukie jukie commented Mar 21, 2023

For environments without write access to /usr/local/bin, the existing function will fail because it attempts to remove the symlink if terraform already exists.
tfswitch should instead check for symlinks at the given binPath or default to user home directory.

lib/install.go Outdated Show resolved Hide resolved
@jukie jukie requested a review from yermulnik March 24, 2023 06:51
@yermulnik
Copy link
Collaborator

This is not related to this PR, though I can see defaultBin definition in lib/install.go and can't make up my mind of why it is (re-)defined here (it is defined in main.go). Am I missing something?

@jukie jukie deleted the branch warrensbox:master March 26, 2023 05:52
@jukie jukie closed this Mar 26, 2023
@jukie jukie deleted the master branch March 26, 2023 05:52
@jukie jukie mentioned this pull request Mar 26, 2023
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.

2 participants