Path separator
Some of the files contain a path for operating system windows while Python 3 on Windows requires this path to use backslash \\
as a path separator. os.path.join("a", "b", "c")
on the other hand is portable and will insert the correct path separator.
See for example file sqpass/install.py, line 11.
Please change from os.path.expanduser("~/AppData/pass.key")
to os.path.expanduser("~\\AppData\\pass.key")
.
Empty value for os.path.expanduser()
See file sqpass/passwordManager/conf.py, line 10.
Instead of os.path.expanduser("")
please use the tilde ~
for windows, too: os.path.expanduser("~")
Maybe even add the same folders as for linux: os.path.expanduser("~\\.config\\sqpass\\")
Use os.makedirs()
sqpass/passwordManager/conf.py, line 31 uses os.mkdir()
which will only create the last directory.
Proposal: use os.makedirs()
to create potentially missing directories on all levels.
Output
Please add the resp. paths for the files and directories reported by sqpass/passwordManager/conf.py
.
Currently:
Passkey created
Config file Found!
Proposal:
Created passkey c:\users\username\AppData\pass.key
Found configuration file c:\users\username\config\sqpass\config.yaml