-
Notifications
You must be signed in to change notification settings - Fork 26
Support root targeting in tuckr push #99
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
base: master
Are you sure you want to change the base?
Support root targeting in tuckr push #99
Conversation
|
@RaphGL I'm thinking of making these functions accept a context object which provides all the required environment variables, especially if we want to implement the |
|
What are you planning to do? But yeah you'll probably need to change some functions in |
|
I do plan on implementing your suggestion for root targeting on Windows, but I still think having a TUCKR_ROOT environment variable would be helpful, if only for testing purposes. It would certainly make testing far more reliable than the current |
|
There's a TUCKR_HOME which has the highest priority for determining where things will be the "root" of your dotfiles. But if you somehow find a better solution, feel free to change it, if it doesn't work out we can roll back. |
|
Checking my own understanding:
|
|
Yes your assumptions are correct. I still don't see how TUCKR_ROOT would be better than |
|
To clarify, I mean TUCKR_ROOT in addition to ^c and the like |
|
So you want to have a TUCKR_ROOT that is used by default if you don't specify a drive on windows? |
|
Not quite: I want a TUCKR_ROOT that specifies what Tuckr should consider to be the root of the file system on all platforms. Detailed examples to make this more clear
Use cases for this
Does this make a little more sense now? Is it a good idea? |
|
Yeah I got it now. I don't really know if it's a good idea tbh, I just don't personally have a use case for this so I don't really care about it. |
|
The main thing I'd definitely use it for is writing a more-thorough test suite. Imo it's probably the best way to get tests that test the entire program including root targeting, since otherwise it would require an Even if there is no publicly accessible environment variable, I'd still want it to be part of Tuckr's context struct, so that tests can provide custom values to it. |
|
give it a shot, it doesn't hurt. we can roll back if it's not a good idea :) |
Work in progress, don't merge yet please!
In particular, I am unsure how to get this working with Windows-style paths, where there is no single file system root.
Resolves #74