Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move boringssl to flutter/third_party. #50601

Merged
merged 11 commits into from
Feb 15, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -315,13 +315,13 @@ deps = {
'src/third_party/googletest':
Var('chromium_git') + '/external/github.com/google/googletest' + '@' + '7f036c5563af7d0329f20e8bb42effb04629f0c0',

'src/third_party/boringssl':
'src/flutter/third_party/boringssl':
matanlurey marked this conversation as resolved.
Show resolved Hide resolved
Var('dart_git') + '/boringssl_gen.git' + '@' + Var('dart_boringssl_gen_rev'),

'src/flutter/third_party/yapf':
Var('flutter_git') + '/third_party/yapf' + '@' + '212c5b5ad8e172d2d914ae454c121c89cccbcb35',

'src/third_party/boringssl/src':
'src/flutter/third_party/boringssl/src':
'https://boringssl.googlesource.com/boringssl.git' + '@' + Var('dart_boringssl_rev'),

'src/third_party/perfetto':
Expand Down
7 changes: 7 additions & 0 deletions build/secondary/third_party/boringssl/BUILD.gn
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
# Copyright 2013 The Flutter Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

group("boringssl") {
public_deps = [ "//flutter/third_party/boringssl" ]
}
4 changes: 2 additions & 2 deletions build/secondary/third_party/libsrtp/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -155,8 +155,8 @@ if (use_system_libsrtp) {
}

if (use_srtp_boringssl) {
deps = [ "//third_party/boringssl:boringssl" ]
public_deps = [ "//third_party/boringssl:boringssl" ]
deps = [ "//flutter/third_party/boringssl:boringssl" ]
public_deps = [ "//flutter/third_party/boringssl:boringssl" ]
sources -= [
"srtp/crypto/cipher/aes_cbc.c",
"srtp/crypto/cipher/aes_icm.c",
Expand Down
2 changes: 1 addition & 1 deletion ci/licenses.sh
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ function verify_licenses() (

local actualLicenseCount
actualLicenseCount="$(tail -n 1 flutter/ci/licenses_golden/licenses_flutter | tr -dc '0-9')"
local expectedLicenseCount=114 # When changing this number: Update the error message below as well describing the newly expected license types.
local expectedLicenseCount=197 # When changing this number: Update the error message below as well describing the newly expected license types.

if [[ $actualLicenseCount -ne $expectedLicenseCount ]]; then
echo "=============================== ERROR ==============================="
Expand Down
780 changes: 405 additions & 375 deletions ci/licenses_golden/excluded_files

Large diffs are not rendered by default.

4,849 changes: 4,771 additions & 78 deletions ci/licenses_golden/licenses_flutter

Large diffs are not rendered by default.

4,787 changes: 2 additions & 4,785 deletions ci/licenses_golden/licenses_third_party

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion ci/licenses_golden/tool_signature
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Signature: 7cfe768e86f1799dbfcbc149e00e4bb4
Signature: f823bf2a897af27f12f1c16a83c3e216

2 changes: 1 addition & 1 deletion common/graphics/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ source_set("graphics") {
"//flutter/shell/common:base64",
"//flutter/shell/version:version",
"//flutter/skia",
"//flutter/third_party/boringssl",
"//flutter/third_party/rapidjson",
"//third_party/boringssl",
]

public_configs = [ "//flutter:config" ]
Expand Down
8 changes: 4 additions & 4 deletions tools/licenses/lib/main.dart
Original file line number Diff line number Diff line change
Expand Up @@ -990,6 +990,7 @@ class _RepositoryDirectory extends _RepositoryEntry implements LicenseSource {
static final RegExp _licenseNamePattern = RegExp(r'^(?!.*\.py$)(?!.*(?:no|update)-copyright)(?!.*mh-bsd-gcc).*\b_*(?:license(?!\.html)|copying|copyright|notice|l?gpl|GPLv2|bsd|mit|mpl?|ftl|Apache)_*\b', caseSensitive: false);

static const Map<String, _Constructor> _specialCaseFiles = <String, _Constructor>{
'/flutter/third_party/boringssl/src/LICENSE': _RepositoryOpenSSLLicenseFile.new,
'/flutter/third_party/inja/third_party/include/nlohmann/json.hpp': _RepositoryInjaJsonFile.new,
'/flutter/third_party/libjpeg-turbo/src/LICENSE': _RepositoryLibJpegTurboLicenseFile.new,
'/flutter/third_party/libjpeg-turbo/src/README.ijg': _RepositoryReadmeIjgFile.new,
Expand All @@ -998,7 +999,6 @@ class _RepositoryDirectory extends _RepositoryEntry implements LicenseSource {
'/flutter/third_party/rapidjson/license.txt': _RepositoryOpaqueLicenseFile.new,
'/fuchsia/sdk/linux/LICENSE.vulkan': _RepositoryFuchsiaSdkLinuxLicenseFile.new,
'/fuchsia/sdk/mac/LICENSE.vulkan': _RepositoryFuchsiaSdkLinuxLicenseFile.new,
'/third_party/boringssl/src/LICENSE': _RepositoryOpenSSLLicenseFile.new,
'/third_party/dart/LICENSE': _RepositoryDartLicenseFile.new,
'/third_party/freetype2/LICENSE.TXT': _RepositoryFreetypeLicenseFile.new,
'/third_party/icu/LICENSE': _RepositoryIcuLicenseFile.new,
Expand Down Expand Up @@ -1440,9 +1440,6 @@ class _RepositoryRootThirdPartyDirectory extends _RepositoryGenericThirdPartyDir

@override
_RepositoryDirectory createSubdirectory(fs.Directory entry) {
if (entry.name == 'boringssl') {
return _RepositoryBoringSSLDirectory(this, entry);
}
if (entry.name == 'expat') {
return _RepositoryExpatDirectory(this, entry);
}
Expand Down Expand Up @@ -1762,6 +1759,9 @@ class _RepositoryFlutterThirdPartyDirectory extends _RepositoryGenericThirdParty

@override
_RepositoryDirectory createSubdirectory(fs.Directory entry) {
if (entry.name == 'boringssl') {
return _RepositoryBoringSSLDirectory(this, entry);
}
if (entry.name == 'expat') {
return _RepositoryExpatDirectory(this, entry);
}
Expand Down
14 changes: 7 additions & 7 deletions tools/licenses/lib/paths.dart
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ final Set<String> skippedPaths = <String>{
r'flutter/prebuilts',
r'flutter/sky/packages/sky_engine/LICENSE',
r'flutter/third_party/benchmark', // only used by tests
r'flutter/third_party/boringssl/src/crypto/err/err_data_generate.go',
r'flutter/third_party/boringssl/src/fuzz', // testing tools, not shipped
r'flutter/third_party/boringssl/src/rust', // rust-related code is not shipped
r'flutter/third_party/boringssl/src/util', // code generators, not shipped
r'flutter/third_party/depot_tools', // only used for the Windows build system.
r'flutter/third_party/expat/expat/doc',
r'flutter/third_party/expat/expat/win32/expat.iss',
Expand Down Expand Up @@ -130,10 +134,6 @@ final Set<String> skippedPaths = <String>{
r'third_party/angle/third_party', // Unused by Flutter: BUILD files with forwarding targets (but no code).
r'third_party/angle/tools', // These are build-time tools, and aren't shipped.
r'third_party/angle/util',
r'third_party/boringssl/src/crypto/err/err_data_generate.go',
r'third_party/boringssl/src/fuzz', // testing tools, not shipped
r'third_party/boringssl/src/rust', // rust-related code is not shipped
r'third_party/boringssl/src/util', // code generators, not shipped
r'third_party/dart/benchmarks', // not shipped in binary
r'third_party/dart/build', // not shipped in binary
r'third_party/dart/docs', // not shipped in binary
Expand Down Expand Up @@ -459,11 +459,11 @@ final List<Pattern> skippedFilePatterns = <Pattern>[
RegExp(r'^flutter/(?:.+/)*[^/]+_unittests?\.[^/]+$'),
RegExp(r'^flutter/lib/web_ui/lib/assets/ahem\.ttf$', expectNoMatch: true), // this gitignored file exists only for testing purposes
RegExp(r'^flutter/sky/packages/sky_engine/LICENSE$'), // that is the output of this script
RegExp(r'^flutter/third_party/boringssl/(?:.+/)*[^/]+_test\.[^/]+$'),
RegExp(r'^flutter/third_party/boringssl/src/crypto/fipsmodule/bn/[^/]+.go$'),
RegExp(r'^flutter/third_party/boringssl/src/crypto/fipsmodule/ec/[^/]+.go$'),
RegExp(r'^third_party/abseil-cpp/(?:.+/)*[^/]+_test\.[^/]+$'),
RegExp(r'^third_party/angle/(?:.+/)*[^/]+_unittest\.[^/]+$'),
RegExp(r'^third_party/boringssl/(?:.+/)*[^/]+_test\.[^/]+$'),
RegExp(r'^third_party/boringssl/src/crypto/fipsmodule/bn/[^/]+.go$'),
RegExp(r'^third_party/boringssl/src/crypto/fipsmodule/ec/[^/]+.go$'),
RegExp(r'^third_party/dart/(?:.+/)*[^/]+_test\.[^/]+$'),
RegExp(r'^third_party/freetype2/docs/(?!FTL\.TXT$).+'), // ignore all documentation except the license
RegExp(r'^third_party/zlib/(?:.+/)*[^/]+_unittest\.[^/]+$'),
Expand Down