Skip to content

compdb: use aspects generated header files#12426

Merged
mattklein123 merged 3 commits intoenvoyproxy:masterfrom
lizan:compdb_header
Aug 3, 2020
Merged

compdb: use aspects generated header files#12426
mattklein123 merged 3 commits intoenvoyproxy:masterfrom
lizan:compdb_header

Conversation

@lizan
Copy link
Copy Markdown
Member

@lizan lizan commented Aug 1, 2020

Clear tech debts solved by grailbio/bazel-compilation-database#58, this should make compilation database generation much faster than before especially when cache doesn't exist.

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

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
lizan added 2 commits July 31, 2020 23:48
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Copy link
Copy Markdown
Member

@dio dio left a comment

Choose a reason for hiding this comment

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

Looks good!

@mattklein123 mattklein123 merged commit 7b8ab2d into envoyproxy:master Aug 3, 2020
Comment on lines +47 to +48
# 2020-07-31
urls = ["https://github.com/grailbio/bazel-compilation-database/archive/c37b909045eb72d29a47f77cc1e9b519dd5c10b6.tar.gz"],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wish I had caught this earlier but this commit is in the latest 0.4.5 release. I'm move back to the release in a subsequent PR.

    bazel_compdb = dict(
        sha256 = "bcecfd622c4ef272fd4ba42726a52e140b961c4eac23025f18b346c968a8cfb4",
        strip_prefix = "bazel-compilation-database-0.4.5",
        urls = ["https://github.com/grailbio/bazel-compilation-database/archive/0.4.5.tar.gz"],
        use_category = ["build"],
    ),

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

rolled in #12449

chaoqin-li1123 pushed a commit to chaoqin-li1123/envoy that referenced this pull request Aug 7, 2020
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
chaoqin-li1123 pushed a commit to chaoqin-li1123/envoy that referenced this pull request Aug 7, 2020
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: chaoqinli <chaoqinli@google.com>
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.

5 participants