-
Notifications
You must be signed in to change notification settings - Fork 5
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
Enable Variable Substitutions for Windows #17
Comments
Hey! We could look at making the substitution and escape character configurable. However, For example, we could allow using I suppose in this situation it would be nicest if the escape character is also just Or.. I could just add another parser that truly works like the normal Windows style, so truly |
Note that you can have variable substitution for windows paths already, you just need to be very explicit: You would have to escape all You can use a forward slash on Windows too, to prevent having to escape all the backslashes. Then you only need to escape the |
Thanks a lot for the swift reply!
I believe that, for the benefit of your repository (which I see as a general-purpose API), both solutions would probably be nice while leaving the default what it is now (i.e., unix-style
I see. I always thought the |
It would be nice to allow variable substitutions for windows machines as well.
Since the '$' is a valid character in windows paths, and these paths use the backslash (
\
) character instead of the forward slash, the default substitution cannot be used. Instead, I would propose the%VARIABLE_NAME%
syntax.Note that I am not a windows user, but a user of this library and it would be nice to offer support for windows as well.
I'm happy to discuss any implementation details and can also offer to work towards enabling this feature.
The text was updated successfully, but these errors were encountered: