Fix reporting or errors addin building and testing

visual_studio/NativeClientVSAddIn/buildbot_run.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file
 
 """Entry point for the AddIn build bot.
 
 Perform build steps and output results using the buildbot
 annootator syntax
 """
 
@@ skipping 16 matching lines @@
 def RunCommand(cmd, env=None):
   Log("Running: %s" % cmd)
   Log("CWD: %s" % os.getcwd())
   if type(cmd) == str:
     cmd = cmd.split()
 
   if sys.platform == 'cygwin':
     # allow bat files in the current working directory to
     # be executed on cygwin as they are on win32
     if not os.path.dirname(cmd[0]) and os.path.exists(cmd[0]):
-      cmd = './' + cmd
+      cmd[0] = './' + cmd[0]
 
-  rtn = subprocess.call(cmd, shell=True, env=env)
+  rtn = subprocess.call(cmd, env=env)
   if rtn:
+    Log("Command returned non-zero exit code: %s" % rtn)
     Log('@@@STEP_FAILURE@@@')
     sys.exit(1)
 
 
 def StepBuild():
   Log('@@@BUILD_STEP build AddIn@@@')
   RunCommand('build.bat')
 
 
 def StepInstall():
@@ skipping 82 matching lines @@
 def main():
   StepBuild()
   StepInstall()
   StepInstallSDK()
   StepTest()
   StepArchive()
 
 
 if __name__ == '__main__':
   main()