Skip to content

build: add a script to setup clang#8682

Merged
lizan merged 9 commits intoenvoyproxy:masterfrom
lizan:clang_setup
Oct 23, 2019
Merged

build: add a script to setup clang#8682
lizan merged 9 commits intoenvoyproxy:masterfrom
lizan:clang_setup

Conversation

@lizan
Copy link
Copy Markdown
Member

@lizan lizan commented Oct 20, 2019

Signed-off-by: Lizan Zhou lizan@tetrate.io

Description:
Add a script to setup clang prebuilt packages.

Risk Level: Low
Testing: local, CI
Docs Changes: bazel/README.md
Release Notes:
Fixes #8625

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
lizan added 2 commits October 21, 2019 16:16
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan lizan marked this pull request as ready for review October 21, 2019 16:26
@lizan lizan requested a review from htuch October 21, 2019 16:27
Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I can see the benefit of pushing more of this into a standard bazelrc flow, a few comments/questions.
/wait

lizan added 2 commits October 22, 2019 03:22
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
lizan added 3 commits October 22, 2019 17:53
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, ready to merge when CI passes, thanks.

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan lizan merged commit 87d7224 into envoyproxy:master Oct 23, 2019
spenceral added a commit to spenceral/envoy that referenced this pull request Oct 23, 2019
* master: (54 commits)
  Update dependencies - Go, Bazel toos, xxHash, nanopb, rules_go, curl, protobuf, Jinja, yaml-cpp, re2 (envoyproxy#8728)
  test: increase coverage of listener_manager_impl.cc (envoyproxy#8737)
  test: modify some macros to reduce number of uncovered lines reported (envoyproxy#8725)
  build: add a script to setup clang (envoyproxy#8682)
  http: fix ssl_redirect on external (envoyproxy#8664)
  docs: update fedora build requirements (envoyproxy#8641)
  fix draining listener removal logs (envoyproxy#8733)
  dubbo: fix router doc (envoyproxy#8734)
  server: provide server health status when stats disabled (envoyproxy#8482)
  router: adding a knob to configure a cap on buffering for shadowing/retries (envoyproxy#8574)
  tcp proxy: add default 1 hour idle timeout (envoyproxy#8705)
  thrift: fix filter names in docs (envoyproxy#8726)
  Quiche changes to avoid gcc flags on Windows (envoyproxy#8514)
  test: increase test coverage in Router::HeaderParser (envoyproxy#8721)
  admin: add drain listeners endpoint  (envoyproxy#8415)
  buffer filter: add content-length when ending stream with trailers (envoyproxy#8609)
  clarify draining option docs (envoyproxy#8712)
  build: ignore go-control-plane mirror git commit error code (envoyproxy#8703)
  api: remove API type database from checked in artifacts. (envoyproxy#8716)
  admin: correct help strings (envoyproxy#8710)
  ...

Signed-off-by: Spencer Lewis <slewis@squareup.com>
derekargueta pushed a commit to derekargueta/envoy that referenced this pull request Oct 24, 2019
Description:
Add a script to setup clang prebuilt packages.

Risk Level: Low
Testing: local, CI
Docs Changes: bazel/README.md
Release Notes:
Fixes envoyproxy#8625

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

build: bring back function and vptr ubsan

2 participants