-
Notifications
You must be signed in to change notification settings - Fork 14
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 support for rtx
#24
Comments
Is it possible to make it work by simply setting an environment variable? |
Yes!
rtx installs under its def create(cls) -> t.Self | None:
- asdf_root = os.path.expanduser(
- os.path.expandvars(os.getenv("ASDF_DATA_DIR", "~/.asdf"))
+ rtx_root = os.path.expanduser(
+ os.path.expandvars(os.getenv("RTX_DATA_DIR", "~/.local/share/rtx"))
)
- if not os.path.exists(asdf_root):
+ if not os.path.exists(rtx_root):
return None
- return cls(Path(asdf_root))
+ return cls(Path(rtx_root)) If you like it, I could come up with a PR. |
I mean to set |
I guess so, though I don't know whether this could create a conflict with other tools (hopefully none, since One more thing about my use case. I found out about Nevertheless, your solution should work so feel free to close the issue! Thanks for your time. |
rtx
is an asdf compatible runtime executor written in rust. Executables are installed by default under~/.local/share/rtx
(ref here) though it can be changed. Would you be interested in findpython supporting rtx too? I could volunteer to work on the PR too.The text was updated successfully, but these errors were encountered: