Skip to content
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.

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

Path map non-direct children of tree artifacts during header discovery #23690

Closed
wants to merge 1 commit into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Sep 20, 2024

Fixes header inclusion errors with path mapping when including a header that is a non-direct child of a tree artifact.

Fixes #23681

Fixes header inclusion errors with path mapping when including a header that is not a direct child of a tree artifact.
@fmeum fmeum requested a review from lberki as a code owner September 20, 2024 08:11
@fmeum
Copy link
Collaborator Author

fmeum commented Sep 20, 2024

@bazel-io fork 7.4.0

@github-actions github-actions bot added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Sep 20, 2024
@fmeum fmeum requested review from comius and removed request for lberki September 20, 2024 08:11
@fmeum
Copy link
Collaborator Author

fmeum commented Sep 20, 2024

FYI @keith

@keith
Copy link
Member

keith commented Sep 20, 2024

verified on our project, thanks!

@comius comius added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Sep 23, 2024
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Sep 24, 2024
@fmeum fmeum deleted the 23681-rfc-path-mapping branch September 24, 2024 10:02
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Sep 24, 2024
Fixes header inclusion errors with path mapping when including a header that is a non-direct child of a tree artifact.

Fixes bazelbuild#23681

Closes bazelbuild#23690.

PiperOrigin-RevId: 678118252
Change-Id: I37f4d5e8827785b291b5655967afba8f2a673df2
github-merge-queue bot pushed a commit that referenced this pull request Sep 26, 2024
…discovery (#23745)

Fixes header inclusion errors with path mapping when including a header
that is a non-direct child of a tree artifact.

Fixes #23681

Closes #23690.

PiperOrigin-RevId: 678118252
Change-Id: I37f4d5e8827785b291b5655967afba8f2a673df2

Commit
9cfffdf

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-CPP Issues for C++ rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing dependency declaration with path mapping
3 participants