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

.terraform-version file: invalid tool version request: latest:^1.3 #917

Closed
nlamirault opened this issue Sep 25, 2023 · 3 comments
Closed

Comments

@nlamirault
Copy link
Contributor

Describe the bug
with a .terraform-version file which contains an entry like: latest:^1.3 we have this error:

❯ cd ../terraform-azurerm-aks
The application panicked (crashed).
Message:  invalid tool version request: latest:^1.3
Location: src/toolset/tool_version_request.rs:47

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.
The application panicked (crashed).
Message:  invalid tool version request: latest:^1.3
Location: src/toolset/tool_version_request.rs:47

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

To Reproduce
Create a .terraform-version file with this content:

cat .terraform-version
latest:^1.3

Expected behavior

rtx doctor output

❯ rtx doctor
The application panicked (crashed).
Message:  invalid tool version request: latest:^1.3
Location: src/toolset/tool_version_request.rs:47

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.
The application panicked (crashed).
Message:  invalid tool version request: latest:^1.3
Location: src/toolset/tool_version_request.rs:47

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.```

**Additional context**
Add any other context about the problem here.
@nlamirault nlamirault added the bug label Sep 25, 2023
@jdx
Copy link
Owner

jdx commented Oct 16, 2023

see here for how to disable legacy file parsing to get around this

the issue is that the file supports semver but rtx does not

@nlamirault
Copy link
Contributor Author

nlamirault commented Oct 16, 2023

thanks @jdx
so Rtx can't manage Terraform version using this syntax ?

latest:^1.3

@jdx
Copy link
Owner

jdx commented Oct 16, 2023

right, rtx doesn't currently support semver

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

No branches or pull requests

2 participants