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.

Sign up for GitHub

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

Jump to bottom

-chdir doesn't work with an absolute path #354

Open
OJFord opened this issue Jul 27, 2022 · 0 comments · May be fixed by #355
Open

-chdir doesn't work with an absolute path #354

OJFord opened this issue Jul 27, 2022 · 0 comments · May be fixed by #355

Comments

@OJFord
Copy link
Contributor

OJFord commented Jul 27, 2022 

$tmpdir="$(mktemp -d)"
$echo 1.2.3 > "$tmpdir/.terraform-version"
$terraform -chdir="$tmpdir" init

Fails with:

cat: /home/ojf/code/tfenv/version: No such file or directory
Version could not be resolved (set by /home/ojf/code/tfenv/version or tfenv use )

The workaround is to ensure it's relative to the current working directory:

$terraform -chdir="$(realpath --relative-to="$PWD" "$tmpdir")" init
OJFord added a commit to OJFord/tfenv that referenced this issue Jul 27, 2022
@OJFord
Add test to reproduce tfutils#354
2bc027d
OJFord added a commit to OJFord/tfenv that referenced this issue Jul 27, 2022
@OJFord
Fix use of -chdir with an absolute path
3e59869 
Fixes tfutils#354.
@OJFord OJFord linked a pull request Jul 27, 2022 that will close this issue
Open
Zordrak pushed a commit to OJFord/tfenv that referenced this issue Oct 1, 2022
@OJFord @Zordrak
Add test to reproduce tfutils#354
23d6cbc
Zordrak pushed a commit to OJFord/tfenv that referenced this issue Oct 1, 2022
@OJFord @Zordrak
Fix use of -chdir with an absolute path
129ce8f 
Fixes tfutils#354.
Zordrak pushed a commit to OJFord/tfenv that referenced this issue Oct 1, 2022
@OJFord @Zordrak
Add test to reproduce tfutils#354
5110017
Zordrak pushed a commit to OJFord/tfenv that referenced this issue Oct 1, 2022
@OJFord @Zordrak
Fix use of -chdir with an absolute path
edde4d2 
Fixes tfutils#354.
OJFord added a commit to OJFord/tfenv that referenced this issue Nov 4, 2022
@OJFord
Add test to reproduce tfutils#354
f4d9f1e
OJFord added a commit to OJFord/tfenv that referenced this issue Nov 4, 2022
@OJFord
Fix use of -chdir with an absolute path
4e3c605 
Fixes tfutils#354.
Stretch96 added a commit to dxw/dalmatian-tools that referenced this issue Nov 6, 2023
@Stretch96
Fix tfenv -chdir usage
ff1a06a 
* When using the `-chdir` flag with terraform, tfenv doesn't respect the
  `.terraform-version` file if the paht is absolute: tfutils/tfenv#354
* This changes the path so that it is relative
@Stretch96 Stretch96 mentioned this issue Nov 6, 2023
Merged
Stretch96 added a commit to dxw/dalmatian-tools that referenced this issue Nov 6, 2023
@Stretch96
Fix tfenv -chdir usage
1d51700 
* When using the `-chdir` flag with terraform, tfenv doesn't respect the
  `.terraform-version` file if the paht is absolute: tfutils/tfenv#354
* This changes the path so that it is relative
Zordrak pushed a commit to OJFord/tfenv that referenced this issue Dec 19, 2023
@OJFord @Zordrak
Add test to reproduce tfutils#354
f8ca49f
Zordrak pushed a commit to OJFord/tfenv that referenced this issue Dec 19, 2023
@OJFord @Zordrak
Fix use of -chdir with an absolute path
242a206 
Fixes tfutils#354.
OJFord added a commit to OJFord/tfenv that referenced this issue Feb 10, 2024
@OJFord
Add test to reproduce tfutils#354
934a002
OJFord added a commit to OJFord/tfenv that referenced this issue Feb 10, 2024
@OJFord
Fix use of -chdir with an absolute path
b5782f8 
Fixes tfutils#354.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix use of -chdir with an absolute path OJFord/tfenv
1 participant
@OJFord