From ced6c7a597324a17c9cb55b11ea90515e8be8201 Mon Sep 17 00:00:00 2001 From: Jackson Gardner Date: Tue, 12 Mar 2024 09:42:13 -0700 Subject: [PATCH 1/3] Reland "Move emscripten out of the buildroot into the flutter repo" --- DEPS | 6 +++--- ci/licenses_golden/excluded_files | 2 +- ci/licenses_golden/tool_signature | 2 +- lib/web_ui/dev/roll_fallback_fonts.dart | 2 +- lib/web_ui/dev/steps/copy_artifacts_step.dart | 1 + tools/activate_emsdk.py | 2 +- tools/licenses/lib/paths.dart | 2 +- 7 files changed, 9 insertions(+), 8 deletions(-) diff --git a/DEPS b/DEPS index 023a10f28e14b..6eaf2d8070dda 100644 --- a/DEPS +++ b/DEPS @@ -283,7 +283,7 @@ allowed_hosts = [ ] deps = { - 'src': 'https://github.com/flutter/buildroot.git' + '@' + 'e9df434539745202fa5f9b729fe5f82e5de3f025', + 'src': 'https://github.com/flutter/buildroot.git' + '@' + '34b081632dcc446d121d97a102f8fac7ee4cb751', 'src/flutter/third_party/depot_tools': Var('chromium_git') + '/chromium/tools/depot_tools.git' + '@' + '580b4ff3f5cd0dcaa2eacda28cefe0f45320e8f7', @@ -829,7 +829,7 @@ deps = { 'dep_type': 'cipd', }, - 'src/buildtools/emsdk': { + 'src/flutter/prebuilts/emsdk': { 'url': Var('skia_git') + '/external/github.com/emscripten-core/emsdk.git' + '@' + 'a896e3d066448b3530dbcaa48869fafefd738f57', 'condition': 'download_emsdk', }, @@ -1055,7 +1055,7 @@ deps = { 'dep_type': 'cipd', }, - 'src/third_party/google_fonts_for_unit_tests': { + 'src/flutter/third_party/google_fonts_for_unit_tests': { 'packages': [ { 'package': 'flutter/flutter_font_fallbacks', diff --git a/ci/licenses_golden/excluded_files b/ci/licenses_golden/excluded_files index cf422f0f6441d..23bd4f19eea1a 100644 --- a/ci/licenses_golden/excluded_files +++ b/ci/licenses_golden/excluded_files @@ -1541,6 +1541,7 @@ ../../../flutter/third_party/glfw/src/CMakeLists.txt ../../../flutter/third_party/glfw/tests ../../../flutter/third_party/gn +../../../flutter/third_party/google_fonts_for_unit_tests ../../../flutter/third_party/googletest ../../../flutter/third_party/gtest-parallel/.git ../../../flutter/third_party/gtest-parallel/.gitignore @@ -3130,7 +3131,6 @@ ../../../third_party/dart/utils/OWNERS ../../../third_party/dart/utils/compiler/.gitignore ../../../third_party/dart/utils/dartanalyzer/.gitignore -../../../third_party/google_fonts_for_unit_tests ../../../third_party/gradle ../../../third_party/java ../../../third_party/libcxx/.clang-format diff --git a/ci/licenses_golden/tool_signature b/ci/licenses_golden/tool_signature index 375dde35c83ce..950f7ddb9e9b8 100644 --- a/ci/licenses_golden/tool_signature +++ b/ci/licenses_golden/tool_signature @@ -1,2 +1,2 @@ -Signature: 6d4a7215ee5679ace5ae272f414246bb +Signature: 2d5041d8bed2f11b22aad4b0e1fafc67 diff --git a/lib/web_ui/dev/roll_fallback_fonts.dart b/lib/web_ui/dev/roll_fallback_fonts.dart index a281138009fda..5981e28d627ef 100644 --- a/lib/web_ui/dev/roll_fallback_fonts.dart +++ b/lib/web_ui/dev/roll_fallback_fonts.dart @@ -323,7 +323,7 @@ OTHER DEALINGS IN THE FONT SOFTWARE. ); await runProcess('gclient', [ 'setdep', - '--revision=src/third_party/google_fonts_for_unit_tests:$packageName@$versionString', + '--revision=src/flutter/third_party/google_fonts_for_unit_tests:$packageName@$versionString', '--deps-file=$depFilePath' ]); } diff --git a/lib/web_ui/dev/steps/copy_artifacts_step.dart b/lib/web_ui/dev/steps/copy_artifacts_step.dart index b02f6f49e4c8b..7c7293bbd5fc6 100644 --- a/lib/web_ui/dev/steps/copy_artifacts_step.dart +++ b/lib/web_ui/dev/steps/copy_artifacts_step.dart @@ -105,6 +105,7 @@ class CopyArtifactsStep implements PipelineStep { final io.Directory fallbackFontsSource = io.Directory(pathlib.join( environment.engineSrcDir.path, + 'flutter', 'third_party', 'google_fonts_for_unit_tests', )); diff --git a/tools/activate_emsdk.py b/tools/activate_emsdk.py index 11d0f302dd64b..64219942a4a97 100644 --- a/tools/activate_emsdk.py +++ b/tools/activate_emsdk.py @@ -9,7 +9,7 @@ import subprocess import sys -EMSDK_ROOT = os.path.abspath(os.path.join(__file__, '..', '..', '..', 'buildtools', 'emsdk')) +EMSDK_ROOT = os.path.abspath(os.path.join(__file__, '..', '..', 'prebuilts', 'emsdk')) EMSDK_PATH = os.path.join(EMSDK_ROOT, 'emsdk.py') diff --git a/tools/licenses/lib/paths.dart b/tools/licenses/lib/paths.dart index 4f40ef3c8ef06..1c7ae072ac737 100644 --- a/tools/licenses/lib/paths.dart +++ b/tools/licenses/lib/paths.dart @@ -63,6 +63,7 @@ final Set skippedPaths = { r'flutter/third_party/glfw/deps', // Only used by examples and tests; not linked in build. r'flutter/third_party/glfw/docs', r'flutter/third_party/gn', + r'flutter/third_party/google_fonts_for_unit_tests', // only used in web unit tests r'flutter/third_party/harfbuzz/docs', r'flutter/third_party/harfbuzz/util', // utils are command line tools that do not end up in the binary r'flutter/third_party/icu/filters', @@ -198,7 +199,6 @@ final Set skippedPaths = { r'third_party/dart/third_party/pkg_tested', r'third_party/dart/third_party/requirejs', // only used by DDC r'third_party/dart/tools', // not shipped in binary - r'third_party/google_fonts_for_unit_tests', // only used in web unit tests r'third_party/gradle', r'third_party/java', // only used for Android builds r'third_party/libcxx/benchmarks', From 258ad75946625027e1d83597497de9d25974e69f Mon Sep 17 00:00:00 2001 From: Jackson Gardner Date: Tue, 12 Mar 2024 10:23:21 -0700 Subject: [PATCH 2/3] Update tool signature. --- ci/licenses_golden/tool_signature | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ci/licenses_golden/tool_signature b/ci/licenses_golden/tool_signature index fd7c613b0e911..950f7ddb9e9b8 100644 --- a/ci/licenses_golden/tool_signature +++ b/ci/licenses_golden/tool_signature @@ -1,2 +1,2 @@ -Signature: 3c449c19152f3b271877be5efd19efa1 +Signature: 2d5041d8bed2f11b22aad4b0e1fafc67 From 2d3809ac90a92a5f61e9fa537dbdbcc1f099c52c Mon Sep 17 00:00:00 2001 From: Jackson Gardner Date: Tue, 12 Mar 2024 10:57:31 -0700 Subject: [PATCH 3/3] Update tool signature again. --- ci/licenses_golden/tool_signature | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ci/licenses_golden/tool_signature b/ci/licenses_golden/tool_signature index 950f7ddb9e9b8..89511c0d45801 100644 --- a/ci/licenses_golden/tool_signature +++ b/ci/licenses_golden/tool_signature @@ -1,2 +1,2 @@ -Signature: 2d5041d8bed2f11b22aad4b0e1fafc67 +Signature: 10ea95d73c5e8a92240f014b15634a8c