From 39f7a10a04a914384944d8bf62621144ac4eeaa3 Mon Sep 17 00:00:00 2001 From: brettw Date: Fri, 2 Sep 2016 14:01:07 -0700 Subject: [PATCH] Move uils and sksl to a common sources GYP file. This is for the coalescing of sources in one GYP file to improve GN runtime. If this is successful, the other sources lists will also be merged into this one file. Inlines "../src" and "../include" for skia_source_dir and skia_include_dir. Evaluating these with GYP's variable expansion is annoying since I think another layer of nesting is required. Coding these explicitly is also more clear. This used to be required because Chromium would reference these .gypi files directly with a different path, but this no longer happens. Removes the chromium defines gypi which is no longer referenced. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2302723005 Review-Url: https://codereview.chromium.org/2302723005 --- gn/shared_sources.gni | 23 ++----- gyp/skia_for_chromium_defines.gypi | 22 ------ gyp/skia_sources.gypi | 136 +++++++++++++++++++++++++++++++++++++ gyp/sksl.gyp | 8 ++- gyp/sksl.gypi | 22 ------ gyp/skslc.gyp | 8 ++- gyp/utils.gyp | 8 +-- gyp/utils.gypi | 120 -------------------------------- 8 files changed, 155 insertions(+), 192 deletions(-) delete mode 100644 gyp/skia_for_chromium_defines.gypi create mode 100644 gyp/skia_sources.gypi delete mode 100644 gyp/sksl.gypi delete mode 100644 gyp/utils.gypi diff --git a/gn/shared_sources.gni b/gn/shared_sources.gni index 8525883..9a4af74 100644 --- a/gn/shared_sources.gni +++ b/gn/shared_sources.gni @@ -69,27 +69,14 @@ _pdf_gypi = exec_script("gypi_to_gn.py", [ "../gyp/pdf.gypi" ]) skia_pdf_sources = _pdf_gypi.sources -# SKSL. -_sksl_gypi = exec_script("gypi_to_gn.py", +_sources_gypi = exec_script("gypi_to_gn.py", [ - rebase_path("../gyp/sksl.gypi"), - "--replace=<(skia_include_path)=$_path_to_include", - "--replace=<(skia_src_path)=$_path_to_src", + rebase_path("../gyp/skia_sources.gypi"), ], "scope", - [ "../gyp/sksl.gypi" ]) -skia_sksl_sources = _sksl_gypi.sources - -# Utils. -_utils_gypi = exec_script("gypi_to_gn.py", - [ - rebase_path("../gyp/utils.gypi"), - "--replace=<(skia_include_path)=$_path_to_include", - "--replace=<(skia_src_path)=$_path_to_src", - ], - "scope", - [ "../gyp/utils.gypi" ]) -skia_utils_sources = _utils_gypi.sources + [ "../gyp/skia_sources.gypi" ]) +skia_sksl_sources = get_path_info(_sources_gypi.sksl_sources, "abspath") +skia_utils_sources = get_path_info(_sources_gypi.utils_sources, "abspath") # Skia Chromium defines. These flags will be defined in chromium If these # become 'permanent', they should be moved into Chrome's skia build file. diff --git a/gyp/skia_for_chromium_defines.gypi b/gyp/skia_for_chromium_defines.gypi deleted file mode 100644 index 6ab5cc8..0000000 --- a/gyp/skia_for_chromium_defines.gypi +++ /dev/null @@ -1,22 +0,0 @@ -# Copyright 2014 Google Inc. -# Use of this source code is governed by a BSD-style license that can be -# found in the LICENSE file. - -# THIS FILE IS OBSOLETE. It will be deleted pending a two-sided update to -# Chromium. Please update the new list in gn/shared_sources.gni. - -# This file is included by chrome's skia/skia_common.gypi, and is intended to -# augment the skia flags that are set there. - -{ - 'variables': { - - # These flags will be defined in chromium - # - # If these become 'permanent', they should be moved into skia_common.gypi - # - 'skia_for_chromium_defines': [ - 'SK_IGNORE_LINEONLY_AA_CONVEX_PATH_OPTS', - ], - }, -} diff --git a/gyp/skia_sources.gypi b/gyp/skia_sources.gypi new file mode 100644 index 0000000..5303652 --- /dev/null +++ b/gyp/skia_sources.gypi @@ -0,0 +1,136 @@ +# Copyright 2015 Google Inc. +# +# Use of this source code is governed by a BSD-style license that can be +# found in the LICENSE file. +# Include this gypi to include all 'utils' files +# The parent gyp/gypi file must define +# 'skia_src_path' e.g. skia/trunk/src +# 'skia_include_path' e.g. skia/trunk/include +# +# The skia build defines these in common_variables.gypi +# +{ + 'utils_sources': [ + '../include/utils/SkBoundaryPatch.h', + '../include/utils/SkFrontBufferedStream.h', + '../include/utils/SkCamera.h', + '../include/utils/SkCanvasStateUtils.h', + '../include/utils/SkDumpCanvas.h', + '../include/utils/SkEventTracer.h', + '../include/utils/SkInterpolator.h', + '../include/utils/SkLayer.h', + '../include/utils/SkMeshUtils.h', + '../include/utils/SkNoSaveLayerCanvas.h', + '../include/utils/SkNWayCanvas.h', + '../include/utils/SkNullCanvas.h', + '../include/utils/SkPaintFilterCanvas.h', + '../include/utils/SkParse.h', + '../include/utils/SkParsePath.h', + '../include/utils/SkPictureUtils.h', + '../include/utils/SkRandom.h', + '../include/utils/SkTextBox.h', + + '../src/utils/SkBase64.cpp', + '../src/utils/SkBase64.h', + '../src/utils/SkBitmapSourceDeserializer.cpp', + '../src/utils/SkBitmapSourceDeserializer.h', + '../src/utils/SkBitSet.h', + '../src/utils/SkBoundaryPatch.cpp', + '../src/utils/SkFrontBufferedStream.cpp', + '../src/utils/SkCamera.cpp', + '../src/utils/SkCanvasStack.h', + '../src/utils/SkCanvasStack.cpp', + '../src/utils/SkCanvasStateUtils.cpp', + '../src/utils/SkCurveMeasure.cpp', + '../src/utils/SkCurveMeasure.h', + '../src/utils/SkDashPath.cpp', + '../src/utils/SkDashPathPriv.h', + '../src/utils/SkDeferredCanvas.cpp', + '../src/utils/SkDumpCanvas.cpp', + '../src/utils/SkEventTracer.cpp', + '../src/utils/SkFloatUtils.h', + '../src/utils/SkInterpolator.cpp', + '../src/utils/SkLayer.cpp', + '../src/utils/SkMatrix22.cpp', + '../src/utils/SkMatrix22.h', + '../src/utils/SkMeshUtils.cpp', + '../src/utils/SkMultiPictureDocument.cpp', + '../src/utils/SkNWayCanvas.cpp', + '../src/utils/SkNullCanvas.cpp', + '../src/utils/SkOSFile.cpp', + '../src/utils/SkPaintFilterCanvas.cpp', + '../src/utils/SkParse.cpp', + '../src/utils/SkParseColor.cpp', + '../src/utils/SkParsePath.cpp', + '../src/utils/SkPatchGrid.cpp', + '../src/utils/SkPatchGrid.h', + '../src/utils/SkPatchUtils.cpp', + '../src/utils/SkPatchUtils.h', + '../src/utils/SkRGBAToYUV.cpp', + '../src/utils/SkRGBAToYUV.h', + '../src/utils/SkShadowPaintFilterCanvas.cpp', + '../src/utils/SkShadowPaintFilterCanvas.h', + '../src/utils/SkTextBox.cpp', + '../src/utils/SkTextureCompressor.cpp', + '../src/utils/SkTextureCompressor.h', + '../src/utils/SkTextureCompressor_Utils.h', + '../src/utils/SkTextureCompressor_ASTC.cpp', + '../src/utils/SkTextureCompressor_ASTC.h', + '../src/utils/SkTextureCompressor_Blitter.h', + '../src/utils/SkTextureCompressor_R11EAC.cpp', + '../src/utils/SkTextureCompressor_R11EAC.h', + '../src/utils/SkTextureCompressor_LATC.cpp', + '../src/utils/SkTextureCompressor_LATC.h', + '../src/utils/SkThreadUtils.h', + '../src/utils/SkThreadUtils_pthread.cpp', + '../src/utils/SkThreadUtils_pthread.h', + '../src/utils/SkThreadUtils_win.cpp', + '../src/utils/SkThreadUtils_win.h', + '../src/utils/SkWhitelistTypefaces.cpp', + + #mac + '../include/utils/mac/SkCGUtils.h', + '../src/utils/mac/SkCreateCGImageRef.cpp', + + #windows + '../src/utils/win/SkAutoCoInitialize.h', + '../src/utils/win/SkAutoCoInitialize.cpp', + '../src/utils/win/SkDWrite.h', + '../src/utils/win/SkDWrite.cpp', + '../src/utils/win/SkDWriteFontFileStream.cpp', + '../src/utils/win/SkDWriteFontFileStream.h', + '../src/utils/win/SkDWriteGeometrySink.cpp', + '../src/utils/win/SkDWriteGeometrySink.h', + '../src/utils/win/SkHRESULT.h', + '../src/utils/win/SkHRESULT.cpp', + '../src/utils/win/SkIStream.h', + '../src/utils/win/SkIStream.cpp', + '../src/utils/win/SkTScopedComPtr.h', + '../src/utils/win/SkWGL.h', + '../src/utils/win/SkWGL_win.cpp', + + #testing + '../src/fonts/SkGScalerContext.cpp', + '../src/fonts/SkGScalerContext.h', + '../src/fonts/SkRandomScalerContext.cpp', + '../src/fonts/SkRandomScalerContext.h', + '../src/fonts/SkTestScalerContext.cpp', + '../src/fonts/SkTestScalerContext.h', + ], + 'sksl_include_dirs': [ + '../include/config', + '../include/core', + '../include/private', + '../src/sksl', + ], + 'sksl_sources': [ + '../src/sksl/SkSLCompiler.cpp', + '../src/sksl/SkSLIRGenerator.cpp', + '../src/sksl/SkSLParser.cpp', + '../src/sksl/SkSLGLSLCodeGenerator.cpp', + '../src/sksl/SkSLSPIRVCodeGenerator.cpp', + '../src/sksl/SkSLUtil.cpp', + '../src/sksl/ir/SkSLSymbolTable.cpp', + '../src/sksl/ir/SkSLType.cpp', + ], +} diff --git a/gyp/sksl.gyp b/gyp/sksl.gyp index 7e0a4f9..609ace9 100644 --- a/gyp/sksl.gyp +++ b/gyp/sksl.gyp @@ -3,14 +3,16 @@ # Use of this source code is governed by a BSD-style license that can be # found in the LICENSE file. { + 'variables': { + 'includes': [ 'skia_sources.gypi' ] + }, 'targets': [ { 'target_name': 'sksl', 'type': 'static_library', 'standalone_static_library': 1, - 'includes': [ - 'sksl.gypi', - ], + 'sources': [ '<@(sksl_sources)' ], + 'include_dirs': [ '<@(sksl_include_dirs)' ], 'defines': [ 'SKIA' ], diff --git a/gyp/sksl.gypi b/gyp/sksl.gypi deleted file mode 100644 index 07e3800..0000000 --- a/gyp/sksl.gypi +++ /dev/null @@ -1,22 +0,0 @@ -# Copyright 2016 Google Inc. -# -# Use of this source code is governed by a BSD-style license that can be -# found in the LICENSE file. -{ - 'include_dirs': [ - '<(skia_include_path)/config', - '<(skia_include_path)/core', - '<(skia_include_path)/private', - '<(skia_src_path)/sksl', - ], - 'sources': [ - '<(skia_src_path)/sksl/SkSLCompiler.cpp', - '<(skia_src_path)/sksl/SkSLIRGenerator.cpp', - '<(skia_src_path)/sksl/SkSLParser.cpp', - '<(skia_src_path)/sksl/SkSLGLSLCodeGenerator.cpp', - '<(skia_src_path)/sksl/SkSLSPIRVCodeGenerator.cpp', - '<(skia_src_path)/sksl/SkSLUtil.cpp', - '<(skia_src_path)/sksl/ir/SkSLSymbolTable.cpp', - '<(skia_src_path)/sksl/ir/SkSLType.cpp', - ], -} diff --git a/gyp/skslc.gyp b/gyp/skslc.gyp index 6e23493..5bd6c39 100644 --- a/gyp/skslc.gyp +++ b/gyp/skslc.gyp @@ -6,14 +6,16 @@ 'includes': [ 'apptype_console.gypi', ], + 'variables': { + 'includes': [ 'skia_sources.gypi' ], + }, 'targets': [ { 'target_name': 'skslc', 'type': 'executable', - 'includes' : [ - 'sksl.gypi', - ], + 'include_dirs': [ '<@(sksl_include_dirs)' ], 'sources': [ + '<@(sksl_sources)', '../src/sksl/SkSLMain.cpp', ], 'configurations': { diff --git a/gyp/utils.gyp b/gyp/utils.gyp index b6ce7fd..12e7836 100644 --- a/gyp/utils.gyp +++ b/gyp/utils.gyp @@ -4,6 +4,9 @@ # found in the LICENSE file. # Gyp for utils. { + 'variables': { + 'includes': [ 'skia_sources.gypi' ], + }, 'targets': [ { 'target_name': 'utils', @@ -14,9 +17,6 @@ 'core.gyp:*', 'etc1.gyp:libetc1', ], - 'includes': [ - 'utils.gypi', - ], 'include_dirs': [ '../include/effects', '../include/gpu', @@ -33,7 +33,7 @@ '../src/utils/win', ], 'sources': [ - 'utils.gypi', # Makes the gypi appear in IDEs (but does not modify the build). + '<@(utils_sources)', # Makes the sources appear in IDEs (but does not modify the build). ], 'conditions': [ [ 'skia_os == "mac"', { diff --git a/gyp/utils.gypi b/gyp/utils.gypi deleted file mode 100644 index bf7f797..0000000 --- a/gyp/utils.gypi +++ /dev/null @@ -1,120 +0,0 @@ -# Copyright 2015 Google Inc. -# -# Use of this source code is governed by a BSD-style license that can be -# found in the LICENSE file. -# Include this gypi to include all 'utils' files -# The parent gyp/gypi file must define -# 'skia_src_path' e.g. skia/trunk/src -# 'skia_include_path' e.g. skia/trunk/include -# -# The skia build defines these in common_variables.gypi -# -{ - 'sources': [ - '<(skia_include_path)/utils/SkBoundaryPatch.h', - '<(skia_include_path)/utils/SkFrontBufferedStream.h', - '<(skia_include_path)/utils/SkCamera.h', - '<(skia_include_path)/utils/SkCanvasStateUtils.h', - '<(skia_include_path)/utils/SkDumpCanvas.h', - '<(skia_include_path)/utils/SkEventTracer.h', - '<(skia_include_path)/utils/SkInterpolator.h', - '<(skia_include_path)/utils/SkLayer.h', - '<(skia_include_path)/utils/SkMeshUtils.h', - '<(skia_include_path)/utils/SkNoSaveLayerCanvas.h', - '<(skia_include_path)/utils/SkNWayCanvas.h', - '<(skia_include_path)/utils/SkNullCanvas.h', - '<(skia_include_path)/utils/SkPaintFilterCanvas.h', - '<(skia_include_path)/utils/SkParse.h', - '<(skia_include_path)/utils/SkParsePath.h', - '<(skia_include_path)/utils/SkPictureUtils.h', - '<(skia_include_path)/utils/SkRandom.h', - '<(skia_include_path)/utils/SkTextBox.h', - - '<(skia_src_path)/utils/SkBase64.cpp', - '<(skia_src_path)/utils/SkBase64.h', - '<(skia_src_path)/utils/SkBitmapSourceDeserializer.cpp', - '<(skia_src_path)/utils/SkBitmapSourceDeserializer.h', - '<(skia_src_path)/utils/SkBitSet.h', - '<(skia_src_path)/utils/SkBoundaryPatch.cpp', - '<(skia_src_path)/utils/SkFrontBufferedStream.cpp', - '<(skia_src_path)/utils/SkCamera.cpp', - '<(skia_src_path)/utils/SkCanvasStack.h', - '<(skia_src_path)/utils/SkCanvasStack.cpp', - '<(skia_src_path)/utils/SkCanvasStateUtils.cpp', - '<(skia_src_path)/utils/SkCurveMeasure.cpp', - '<(skia_src_path)/utils/SkCurveMeasure.h', - '<(skia_src_path)/utils/SkDashPath.cpp', - '<(skia_src_path)/utils/SkDashPathPriv.h', - '<(skia_src_path)/utils/SkDeferredCanvas.cpp', - '<(skia_src_path)/utils/SkDumpCanvas.cpp', - '<(skia_src_path)/utils/SkEventTracer.cpp', - '<(skia_src_path)/utils/SkFloatUtils.h', - '<(skia_src_path)/utils/SkInterpolator.cpp', - '<(skia_src_path)/utils/SkLayer.cpp', - '<(skia_src_path)/utils/SkMatrix22.cpp', - '<(skia_src_path)/utils/SkMatrix22.h', - '<(skia_src_path)/utils/SkMeshUtils.cpp', - '<(skia_src_path)/utils/SkMultiPictureDocument.cpp', - '<(skia_src_path)/utils/SkNWayCanvas.cpp', - '<(skia_src_path)/utils/SkNullCanvas.cpp', - '<(skia_src_path)/utils/SkOSFile.cpp', - '<(skia_src_path)/utils/SkPaintFilterCanvas.cpp', - '<(skia_src_path)/utils/SkParse.cpp', - '<(skia_src_path)/utils/SkParseColor.cpp', - '<(skia_src_path)/utils/SkParsePath.cpp', - '<(skia_src_path)/utils/SkPatchGrid.cpp', - '<(skia_src_path)/utils/SkPatchGrid.h', - '<(skia_src_path)/utils/SkPatchUtils.cpp', - '<(skia_src_path)/utils/SkPatchUtils.h', - '<(skia_src_path)/utils/SkRGBAToYUV.cpp', - '<(skia_src_path)/utils/SkRGBAToYUV.h', - '<(skia_src_path)/utils/SkShadowPaintFilterCanvas.cpp', - '<(skia_src_path)/utils/SkShadowPaintFilterCanvas.h', - '<(skia_src_path)/utils/SkTextBox.cpp', - '<(skia_src_path)/utils/SkTextureCompressor.cpp', - '<(skia_src_path)/utils/SkTextureCompressor.h', - '<(skia_src_path)/utils/SkTextureCompressor_Utils.h', - '<(skia_src_path)/utils/SkTextureCompressor_ASTC.cpp', - '<(skia_src_path)/utils/SkTextureCompressor_ASTC.h', - '<(skia_src_path)/utils/SkTextureCompressor_Blitter.h', - '<(skia_src_path)/utils/SkTextureCompressor_R11EAC.cpp', - '<(skia_src_path)/utils/SkTextureCompressor_R11EAC.h', - '<(skia_src_path)/utils/SkTextureCompressor_LATC.cpp', - '<(skia_src_path)/utils/SkTextureCompressor_LATC.h', - '<(skia_src_path)/utils/SkThreadUtils.h', - '<(skia_src_path)/utils/SkThreadUtils_pthread.cpp', - '<(skia_src_path)/utils/SkThreadUtils_pthread.h', - '<(skia_src_path)/utils/SkThreadUtils_win.cpp', - '<(skia_src_path)/utils/SkThreadUtils_win.h', - '<(skia_src_path)/utils/SkWhitelistTypefaces.cpp', - - #mac - '<(skia_include_path)/utils/mac/SkCGUtils.h', - '<(skia_src_path)/utils/mac/SkCreateCGImageRef.cpp', - - #windows - '<(skia_src_path)/utils/win/SkAutoCoInitialize.h', - '<(skia_src_path)/utils/win/SkAutoCoInitialize.cpp', - '<(skia_src_path)/utils/win/SkDWrite.h', - '<(skia_src_path)/utils/win/SkDWrite.cpp', - '<(skia_src_path)/utils/win/SkDWriteFontFileStream.cpp', - '<(skia_src_path)/utils/win/SkDWriteFontFileStream.h', - '<(skia_src_path)/utils/win/SkDWriteGeometrySink.cpp', - '<(skia_src_path)/utils/win/SkDWriteGeometrySink.h', - '<(skia_src_path)/utils/win/SkHRESULT.h', - '<(skia_src_path)/utils/win/SkHRESULT.cpp', - '<(skia_src_path)/utils/win/SkIStream.h', - '<(skia_src_path)/utils/win/SkIStream.cpp', - '<(skia_src_path)/utils/win/SkTScopedComPtr.h', - '<(skia_src_path)/utils/win/SkWGL.h', - '<(skia_src_path)/utils/win/SkWGL_win.cpp', - - #testing - '<(skia_src_path)/fonts/SkGScalerContext.cpp', - '<(skia_src_path)/fonts/SkGScalerContext.h', - '<(skia_src_path)/fonts/SkRandomScalerContext.cpp', - '<(skia_src_path)/fonts/SkRandomScalerContext.h', - '<(skia_src_path)/fonts/SkTestScalerContext.cpp', - '<(skia_src_path)/fonts/SkTestScalerContext.h', - ], -} -- 2.7.4