-
Notifications
You must be signed in to change notification settings - Fork 6
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
| kvstorebackup
fails with defaults on Windows due to invalid path
#9
Comments
The problem occurs because of the fact that when no default is set there is some special construction done to generate the default path as can be observed at https://github.com/deductiv/kvstore_tools/blob/master/bin/kvstore_backup.py#L119-L125 On Windows this results in the following due to not evaluating the value of
Suggested change is to wrap the default_path with
|
Perhaps it is even better not to do any massaging and just take the path verbatim from the configuration file. I fail to see why the splitting and merging is done and I noticed the os.path.expandvars is done later on a more general code path at https://github.com/deductiv/kvstore_tools/blob/master/bin/kvstore_backup.py#L128 |
While using the default values from defaults/kvstore_config.conf
a default backup w/o parameters fails with the following error:
But it does work when manually specifying it as path parameter:
Contents of defaults/kvstore_config.
It seems some internal setting on windows fails to evaluate properly IMHO
The text was updated successfully, but these errors were encountered: