From 8d405330eeb333775156604f7ec55a00768885d7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Mon, 3 Jul 2017 09:14:27 +0200 Subject: [PATCH] Better platform detection. The sys.platform string is 'win32' even on 64-bit Windows. Furthermore, we expect 'windows', not 'win32'. platform.system().lower() gives us this. --- blender_cloud/flamenco/sdk.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/blender_cloud/flamenco/sdk.py b/blender_cloud/flamenco/sdk.py index 275ffe8..dbe745b 100644 --- a/blender_cloud/flamenco/sdk.py +++ b/blender_cloud/flamenco/sdk.py @@ -11,7 +11,7 @@ class Manager(List, Find): @functools.lru_cache() def _sorted_path_replacements(self) -> list: - import sys + import platform if self.path_replacement is None: return [] @@ -23,9 +23,10 @@ class Manager(List, Find): def by_length(item): return -len(item[0]), item[0] - platform = sys.platform - return [(varname, platform_replacements[platform]) - for varname, platform_replacements in sorted(items, key=by_length)] + this_platform = platform.system().lower() + return [(varname, platform_replacements[this_platform]) + for varname, platform_replacements in sorted(items, key=by_length) + if this_platform in platform_replacements] def replace_path(self, some_path: pathlib.PurePath) -> str: """Performs path variable replacement.