Skip to content

Support triggering Makefiles from outside the tree #39344

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

Merged
merged 1 commit into from
May 9, 2025

Conversation

joestringer
Copy link
Member

It can be useful to run make files from other directories, for instance
to build all binaries outside the source tree. As a step to support
this, update direct paths in the Makefile to refer based on the path of
the Makefile itself rather than the directory where make is executed.

It can be useful to run make files from other directories, for instance
to build all binaries outside the source tree. As a step to support
this, update direct paths in the Makefile to refer based on the path of
the Makefile itself rather than the directory where make is executed.

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested review from a team as code owners May 5, 2025 22:20
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label May 5, 2025
@joestringer joestringer requested a review from a team as a code owner May 5, 2025 22:20
@joestringer joestringer requested review from rolinh and derailed May 5, 2025 22:20
@github-actions github-actions bot added the hubble-cli PRs or GitHub issues related with hubble-cli label May 5, 2025
@joestringer
Copy link
Member Author

/test

@joestringer joestringer enabled auto-merge May 5, 2025 22:34
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 9, 2025
@joestringer joestringer added this pull request to the merge queue May 9, 2025
Merged via the queue into main with commit 9695b59 May 9, 2025
403 of 409 checks passed
@joestringer joestringer deleted the pr/joe/tidy-makefiles branch May 9, 2025 22:38
sayboras added a commit that referenced this pull request Jul 1, 2025
Relates: #39344

Signed-off-by: Tam Mach <tam.mach@cilium.io>
github-merge-queue bot pushed a commit that referenced this pull request Jul 1, 2025
Relates: #39344

Signed-off-by: Tam Mach <tam.mach@cilium.io>
rabelmervin pushed a commit to rabelmervin/cilium that referenced this pull request Aug 18, 2025
Relates: cilium#39344

Signed-off-by: Tam Mach <tam.mach@cilium.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hubble-cli PRs or GitHub issues related with hubble-cli ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants