backslash should not be path-sep when using swiftclient under windows

[Dirk Petersen]

When I use python-swiftclient under windows (v. 2.60 Python 3.4 x64), the path separator for pseudo folders is a backslash

muh\viostor\w8.1\amd64\viostor.sys
muh\viostor\w8.1\x86\viostor.cat
muh\viostor\w8.1\x86\viostor.inf
muh\viostor\w8.1\x86\viostor.pdb
muh\viostor\w8.1\x86\viostor.sys
muh\viostor\w8\amd64\viostor.cat
muh\viostor\w8\amd64\viostor.inf
muh\viostor\w8\amd64\viostor.pdb

I am not sure if this behavior was intended but it will break every other GUI client trying to access the data uploaded using the python swift client.

perhaps os.sep should be set to '/' when initially walking through the file system.

[Dirk Petersen (q-petersen)]

os.sep or os.path.sep appear to be read only?

this hack works for me but there may be more elegant ways:

https://github.com/FredHutch/python-swiftclient/commit/57ecf5034b1f58566fe06b6ddfde3d932dbf207b

[Matt Hanlon (matthanlon)]

I don't think this bug involves needing to change the path separator. Rather paths for upload should not be constructed with os.join, they should always use "/" as the separator even on windows.

This issue renders python-swiftclient command line rather useless on windows.

[Christian Schwede (cschwede)]

There is already a patch up for review: https://review.openstack.org/#/c/241064/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/281700

[Dirk Petersen (q-petersen)]

Any desire to move this one along. Just tested the latest swift client and it seems my ol hack from last year is still required.