Skip to content

Fixed issues with --features=external_include_paths #243

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.

Sign up for GitHub

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

Merged
Szelethus merged 6 commits into from
Jul 1, 2026
Merged

Fixed issues with --features=external_include_paths #243

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
bd2bd1c
Fixed issues with --features=external_include_paths
furtib Jun 16, 2026
4c1a3b9
Ignore improvements for Bazel 6
furtib Jun 16, 2026
21bd5fd
Fix pylint
furtib Jun 16, 2026
c832ad5
Enable tests
furtib Jun 16, 2026
44387c4
Remove bazel 6 specific modifications
furtib Jun 16, 2026
bfc52de
Remove quote includes from transitive targets
furtib Jun 30, 2026
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
12 changes: 12 additions & 0 deletions src/compile_commands.bzl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ _cc_rules = [

SYSTEM_INCLUDE = "-isystem "
QUOTE_INCLUDE = "-iquote "
EXTERNAL_INCLUDE = "-isystem "

# Function copied from https://gist.github.com/oquenchil/7e2c2bd761aa1341b458cc25608da50c
# NOTE: added local_defines
Expand Down Expand Up @@ -111,6 +112,11 @@ def get_compile_flags(ctx, dep):
if len(quote_include) == 0:
quote_include = "."
options.append(QUOTE_INCLUDE + quote_include)
if hasattr(compilation_context, "external_includes"):
for external_include in compilation_context.external_includes.to_list():
if len(external_include) == 0:
external_include = "."
options.append(EXTERNAL_INCLUDE + external_include)

for attr in SOURCE_ATTR:
if not hasattr(ctx.rule.attr, attr):
Expand All @@ -135,6 +141,12 @@ def get_compile_flags(ctx, dep):
system_include = "."
options.append(SYSTEM_INCLUDE + system_include)

if hasattr(compilation_context, "external_includes"):
for external_include in compilation_context.external_includes.to_list():
if len(external_include) == 0:
external_include = "."
options.append(EXTERNAL_INCLUDE + external_include)

return options

def get_sources(ctx):
Expand Down
8 changes: 3 additions & 5 deletions test/unit/external_repository/test_external_repo.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class TestImplDepExternalDep(TestBase):
__test_path__ = os.path.dirname(os.path.abspath(__file__))
BAZEL_BIN_DIR = os.path.join("bazel-bin")
BAZEL_TESTLOGS_DIR = os.path.join("bazel-testlogs")
BAZEL_VERSION = None
BAZEL_VERSION: str = ""
Comment thread
Szelethus marked this conversation as resolved.

@final
@classmethod
Expand Down Expand Up @@ -152,8 +152,7 @@ def test_codechecker_external_include_paths(self):
"--experimental_cc_implementation_deps --enable_bzlmod "
"--features=external_include_paths"
)
# FIXME: Should find nothing.h and finish with exit code 0
self.assertEqual(ret, 1, stderr)
self.assertEqual(ret, 0, stderr)

def test_per_file_external_include_paths(self):
"""
Expand All @@ -168,8 +167,7 @@ def test_per_file_external_include_paths(self):
"--experimental_cc_implementation_deps --enable_bzlmod "
"--features=external_include_paths"
)
# FIXME: Should find nothing.h and finish with exit code 0
self.assertEqual(ret, 3, stderr)
self.assertEqual(ret, 0, stderr)


if __name__ == "__main__":
Expand Down
Loading