From c6df9e9f56ec04a27a67336109b276bcb94c4e9e Mon Sep 17 00:00:00 2001 From: Jenny Tam Date: Tue, 18 Jun 2019 12:48:56 -0700 Subject: [PATCH] Updated the comments --- buildscripts/buildtools.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/buildscripts/buildtools.py b/buildscripts/buildtools.py index 1173cb3e2..17713f242 100644 --- a/buildscripts/buildtools.py +++ b/buildscripts/buildtools.py @@ -81,10 +81,10 @@ def determine_compiler(self, sdk_dir, vs_ver): # 15.9.28307.344 # But if only VS 2017 is present, temp.txt will only have one value like this: # 15.9.28307.344 - # On the other hand, if only VS 2019 is present, temp.txt will be empty. + # Likewise, if only VS 2019 is present, temp.txt contains only the one for 16. # We can achieve the above by checking for version [15,16), in which case # even if both compilers are present, it only returns one. If only VS 2019 - # exists, it's empty. + # exists, temp.txt is empty command = '{0} -version [{1},{2}) -property installationVersion '.format(vswhere, vs_ver, vs_ver + 1) os.system(command + ' > temp.txt') @@ -95,7 +95,7 @@ def determine_compiler(self, sdk_dir, vs_ver): vc = ver[:2] if vc == '15': return 'vc15' - else: + else: # For VS2019, it's 'vs' instead of 'vc' return 'vs16' def compiler_version(self, sdk_dir):