From ed72804638c9b240477c5235d72c3823483813b2 Mon Sep 17 00:00:00 2001 From: Hao Wu Date: Thu, 29 Sep 2016 22:47:38 +0800 Subject: BaseTools Build: Fix build break for clean target in Linux In Linux, Command needs to be String instead of list when Command run as shell with True. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao Reviewed-by: Yonghong Zhu --- BaseTools/Source/Python/build/build.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'BaseTools/Source') diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py index b003c67403..f17b55c4cf 100644 --- a/BaseTools/Source/Python/build/build.py +++ b/BaseTools/Source/Python/build/build.py @@ -265,10 +265,10 @@ def LaunchCommand(Command, WorkingDir): # It could be a string or sequence. We find that if command is a string in following Popen(), # ubuntu may fail with an error message that the command is not found. # So here we may need convert command from string to list instance. - if not isinstance(Command, list): - if platform.system() != 'Windows': + if platform.system() != 'Windows': + if not isinstance(Command, list): Command = Command.split() - Command = ' '.join(Command) + Command = ' '.join(Command) Proc = None EndOfProcedure = None -- cgit v1.2.3