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

bpf/Makefile: remove gen_compile_commands make target #29611

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

ti-mo
Copy link
Contributor

@ti-mo ti-mo commented Dec 4, 2023

The presence of the GitHub project URL for Bear got flagged in a GPL license
check. Remove the gen_compile_commands make target as it simply resolves to
bear --make. It doesn't seem to be referenced by any GH workflows, so it
could be invoked by personal scripts/configuration instead.

@ldelossa Could you live with this?

@ti-mo ti-mo requested a review from ldelossa December 4, 2023 15:31
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 4, 2023
@ti-mo ti-mo marked this pull request as ready for review December 4, 2023 15:31
@ti-mo ti-mo requested a review from a team as a code owner December 4, 2023 15:31
@ldelossa
Copy link
Contributor

ldelossa commented Dec 4, 2023

Yup, this was here purely for convenience, didnt realize it would flag a linter. Feel free to removal.

@ldelossa
Copy link
Contributor

ldelossa commented Dec 4, 2023

Spoke with @ti-mo out of band.
It seems OK to simply remove the URL instead of the entire target.
Will force push this change.

The presence of the GitHub project URL for Bear got flagged in a GPL license
check. Remove the bear URL from gen_compile_commands make target so the
GPL license check can pass.

Signed-off-by: Louis DeLosSantos <louis.delos@isovalent.com>
@ldelossa ldelossa added the release-note/misc This PR makes changes that have no direct user impact. label Dec 4, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 4, 2023
@ti-mo ti-mo added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 5, 2023
@ti-mo ti-mo enabled auto-merge December 5, 2023 11:31
@ti-mo ti-mo disabled auto-merge December 5, 2023 11:31
@ti-mo ti-mo merged commit 34ade5c into cilium:main Dec 5, 2023
42 of 43 checks passed
@ti-mo ti-mo deleted the tb/bpf-remove-bear branch December 5, 2023 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

2 participants