Skip to content

Commit

Permalink
build: Remove chromium_url (#12023)
Browse files Browse the repository at this point in the history
This removes chromium_url in favor of GURL.

Risk Level: Low
Testing: Existing tests
Docs Changes: N/A
Release Notes: N/A

Fixes: #6588

Signed-off-by: Dhi Aurrahman <[email protected]>
  • Loading branch information
dio authored Jul 10, 2020
1 parent 706d976 commit d828958
Show file tree
Hide file tree
Showing 17 changed files with 12 additions and 1,377 deletions.
2 changes: 1 addition & 1 deletion .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ build:coverage --strategy=CoverageReport=sandboxed,local
build:coverage --experimental_use_llvm_covmap
build:coverage --collect_code_coverage
build:coverage --test_tag_filters=-nocoverage
build:coverage --instrumentation_filter="//source(?!/common/chromium_url|/extensions/quic_listeners/quiche/platform)[/:],//include[/:]"
build:coverage --instrumentation_filter="//source(?!/extensions/quic_listeners/quiche/platform)[/:],//include[/:]"
coverage:test-coverage --test_arg="-l trace"
coverage:fuzz-coverage --config=asan-fuzzer
coverage:fuzz-coverage --run_under=@envoy//bazel/coverage:fuzz_coverage_wrapper.sh
Expand Down
8 changes: 1 addition & 7 deletions ci/run_clang_tidy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -50,19 +50,13 @@ function exclude_testdata() {
grep -v tools/testdata/check_format/
}

# Do not run clang-tidy against Chromium URL import, this needs to largely
# reflect the upstream structure.
function exclude_chromium_url() {
grep -v source/common/chromium_url/
}

# Exclude files in third_party which are temporary forks from other OSS projects.
function exclude_third_party() {
grep -v third_party/
}

function filter_excludes() {
exclude_testdata | exclude_chromium_url | exclude_win32_impl | exclude_macos_impl | exclude_third_party
exclude_testdata | exclude_win32_impl | exclude_macos_impl | exclude_third_party
}

if [[ -z "${DIFF_REF}" && "${BUILD_REASON}" != "PullRequest" ]]; then
Expand Down
28 changes: 0 additions & 28 deletions source/common/chromium_url/BUILD

This file was deleted.

27 changes: 0 additions & 27 deletions source/common/chromium_url/LICENSE

This file was deleted.

16 changes: 0 additions & 16 deletions source/common/chromium_url/README.md

This file was deleted.

17 changes: 0 additions & 17 deletions source/common/chromium_url/envoy_shim.h

This file was deleted.

16 changes: 0 additions & 16 deletions source/common/chromium_url/url_canon.cc

This file was deleted.

186 changes: 0 additions & 186 deletions source/common/chromium_url/url_canon.h

This file was deleted.

Loading

0 comments on commit d828958

Please sign in to comment.