some fixes in subprocess.Popen args for windows/linux support
This commit is contained in:
parent
91e886163e
commit
6e6f88c336
|
@ -106,7 +106,7 @@ class Media(object):
|
||||||
|
|
||||||
source = self.source
|
source = self.source
|
||||||
process_output = subprocess.Popen(
|
process_output = subprocess.Popen(
|
||||||
[exiftool, source],
|
'%s "%s"' % (exiftool, source),
|
||||||
stdout=subprocess.PIPE,
|
stdout=subprocess.PIPE,
|
||||||
shell=True,
|
shell=True,
|
||||||
universal_newlines=True
|
universal_newlines=True
|
||||||
|
|
|
@ -156,7 +156,7 @@ class Video(Media):
|
||||||
|
|
||||||
source = self.source
|
source = self.source
|
||||||
process_output = subprocess.Popen(
|
process_output = subprocess.Popen(
|
||||||
[exiftool, source],
|
'%s "%s"' % (exiftool, source),
|
||||||
stdout=subprocess.PIPE,
|
stdout=subprocess.PIPE,
|
||||||
shell=True,
|
shell=True,
|
||||||
universal_newlines=True
|
universal_newlines=True
|
||||||
|
|
|
@ -50,7 +50,7 @@ def test_create_directory_invalid_permissions():
|
||||||
if os.name == 'nt':
|
if os.name == 'nt':
|
||||||
raise SkipTest("It isn't implemented on Windows")
|
raise SkipTest("It isn't implemented on Windows")
|
||||||
filesystem = FileSystem()
|
filesystem = FileSystem()
|
||||||
status = filesystem.create_directory(os.path.join('apathwhichdoesnotexist','afolderwhichdoesnotexist'))
|
status = filesystem.create_directory('/apathwhichdoesnotexist/afolderwhichdoesnotexist')
|
||||||
|
|
||||||
assert status == False
|
assert status == False
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue