-
Notifications
You must be signed in to change notification settings - Fork 13
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
setup.py: FileNotFoundError: torfcli/__vars__.py #7
Comments
Thanks for the heads up. Should be fixed now. |
Thanks, @rndusr ! Not sure if it was already there before this last batch of commits, but great stuff the configuration file and the profiles ! Will cut down my script using them. |
It was already there, but it wasn't quite complete.
And it's still not perfect. Originally, the idea was to allow arbitrary options
in the config file, both in the global options and in profiles. That would
include --config and --profile, so you could load a profile from a profile or
even load a different config file and load more profiles from there and so on.
But I couldn't get it to work correctly, and I'm not even sure there's a use
case for complex setups like that. Would you find that useful?
|
I doubt such complex setup will be used. Having global and flat profile options is more than enough, I think. One thing that I did miss though was being able to get values of variables from the environment. I used to set the comment to |
Okay, thanks for your opinion. Looks like I'm feature creeping again.
I like your idea regarding environment variables. Feel free to open an issue on that one.
|
Had never heard the expression ! And yeah, you were ! |
torfcli/_vars_.py was renamed to torfcli/_vars.py but setup.py is still referencing torfcli/_vars_.py:
Running 435d328.
The text was updated successfully, but these errors were encountered: