Support SKIA_OUT on Mac and ChromeOS

gyp_skia

 #!/usr/bin/python
 
 # Copyright 2011 The Android Open Source Project
 #
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # This script is a wrapper which invokes gyp with the correct --depth argument,
 # and supports the automatic regeneration of build files if all.gyp is
 # changed (Linux-only).
@@ skipping 50 matching lines @@
 
 # Return the directory where all the build files are to be written.
 def get_output_dir():
   # SKIA_OUT can be any directory either as a child of the standard out/
   # directory or any given location on the file system (e.g. /tmp/skia)
   output_dir = os.getenv('SKIA_OUT')
 
   if not output_dir:
     return os.path.join(os.path.abspath(script_dir), 'out')
 
-  if (sys.platform.startswith('darwin') and
-      (not os.getenv(ENVVAR_GYP_GENERATORS) or
-       'xcode' in os.getenv(ENVVAR_GYP_GENERATORS))):
-    print 'ERROR: variable SKIA_OUT is not valid on Mac (using xcodebuild)'

[borenet, 2015/05/04 13:46:27]

This actually does work, probably due to our use of ninja instead of xcodebuild
directly.

[djsollen, 2015/05/04 14:03:44]

Will this break the workflow of those who use xcode to build on macs?  As far as
I know not everyone uses ninja.

-    sys.exit(-1);
-
   if os.path.isabs(output_dir):
     return output_dir
   else:
     return os.path.join(os.path.abspath(script_dir), output_dir)
 
 
 if __name__ == '__main__':
   args = sys.argv[1:]
 
   if not os.getenv(ENVVAR_GYP_GENERATORS):
@@ skipping 79 matching lines @@
     if res:
       sys.exit(res)
 
   # This code is copied from Chrome's build/gyp_chromium. It's not clear why
   # the *_runtime variables are reversed.
   if vs2013_runtime_dll_dirs:
     x64_runtime, x86_runtime = vs2013_runtime_dll_dirs
     vs_toolchain.CopyVsRuntimeDlls(
         os.path.join(os.getenv('CHROME_PATH'), get_output_dir()),
         (x86_runtime, x64_runtime))