Skip to content

ggt: make utils.Table imports and CMake dependencies explicit - #6986

Open
rjodinchr wants to merge 1 commit into
KhronosGroup:mainfrom
rjodinchr:ggt-explicit-deps
Open

rjodinchr wants to merge 1 commit into
KhronosGroup:mainfrom
rjodinchr:ggt-explicit-deps

Conversation

@rjodinchr

Copy link
Copy Markdown
Contributor

Append the repository root directory (rather than utils/) to sys.path in utils/ggt.py and import from utils.Table.* so the package path matches the repository layout in hermetic build systems (Bazel, Soong).

Also list the utils/Table/*.py files in the DEPENDS list of the core_tables_body.inc / core_tables_header.inc custom command in source/CMakeLists.txt to match BUILD.bazel and BUILD.gn.

Append the repository root directory (rather than utils/) to sys.path in
utils/ggt.py and import from utils.Table.* so the package path matches
the repository layout in hermetic build systems (Bazel, Soong).

Also list the utils/Table/*.py files in the DEPENDS list of the
core_tables_body.inc / core_tables_header.inc custom command in
source/CMakeLists.txt to match BUILD.bazel and BUILD.gn.

Change-Id: Iafa92de3a42720c7fd6ebf0e405166ed8e55e598
@rjodinchr

Copy link
Copy Markdown
Contributor Author

@dneto0 could you please review?

@alan-baker
alan-baker requested a review from dneto0 October 21, 2026 19:23
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.

1 participant