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: Dual-license code as GPL 2.0 and 2-Clause BSD #1648

Merged
merged 2 commits into from
Nov 1, 2023
Merged

Conversation

kkourt
Copy link
Contributor

@kkourt kkourt commented Oct 24, 2023

As is the case with Cilium, this PR aims to change the license of Tetragon's bpf code to be dual licensed as GPL 2.0 and 2-Clause BSD.

Fixes: #420.

For more context see: cilium/cilium#18823.

As such, I would like to ask all authors of Tetragon bpf code to review this PR.

CC: @jrfastab @olsajiri @tixxdz @tpapagian @kevsecurity @Forsworns @willfindlay @mtardy @sarahfujimori @dmitris @Y-dc

@kkourt kkourt force-pushed the pr/kkourt/bpf-dual branch from a3adc8f to e27112c Compare October 24, 2023 14:08
@kkourt kkourt added the release-note/misc This PR makes changes that have no direct user impact. label Oct 24, 2023
@kkourt kkourt marked this pull request as ready for review October 24, 2023 14:21
@kkourt kkourt requested a review from a team as a code owner October 24, 2023 14:21
Copy link
Contributor

@willfindlay willfindlay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing permissive licensing

@sharlns sharlns self-requested a review October 25, 2023 08:36
Relicense most code under the "bpf/" directory as dual-licensed under
the terms of both the GPL 2.0 license and the 2-Clause BSD license.

Signed-off-by: Kornilios Kourtis <[email protected]>
Relicense most code under the "bpf/" directory as dual-licensed under
the terms of both the GPL 2.0 license and the 2-Clause BSD license.

The files were processed with a sed script, as follows:

    $ cat license.sed
    # C sources and sed scripts that are GPL-2.0
    s=^\(\(#\|//\|/\*\) *SPDX-License-Identifier: \)GPL-2.0\(\( \*/\)\?\)$=\1(GPL-2.0-only OR BSD-2-Clause)\3=
    # License section for ELF objects
    s=\(__attribute__((section("license"), used)).*\)"GPL"=\1"Dual BSD/GPL"=

    $ find bpf -type f -print0 | xargs -0 -P8 sed -i -f ./license.sed

Files in bpf/include were left intact.

The identified contributors, from whom we received no answers to this
date, but whose contributions to bpf/ are only of minor nature such as
straightforward cleanups, small fixes, or spelling corrections, are:
 - Dmitry Savintsev <[email protected]>
 - dechengyuan <[email protected]>

Acked-by: Jiri Olsa <[email protected]>
Acked-by: Djalal Harouni <[email protected]>
Acked-by: Anastasios Papagiannis <[email protected]>
Acked-by: Kevin Sheldrake <[email protected]>
Acked-by: Kornilios Kourtis <[email protected]>
Acked-by: Peihao Yang <[email protected]>
Acked-by: William Findlay <[email protected]>
Acked-by: John Fastabend <[email protected]>
Acked-by: Mahe Tardy <[email protected]>
Acked-by: Sarah Fujimori <[email protected]>
Acked-by: William Findlay <[email protected]>
Signed-off-by: Kornilios Kourtis <[email protected]>
@kkourt kkourt force-pushed the pr/kkourt/bpf-dual branch from e27112c to 851c304 Compare November 1, 2023 07:23
@kkourt
Copy link
Contributor Author

kkourt commented Nov 1, 2023

Dead link was already fixed, merging.

@kkourt kkourt merged commit 39cf6ff into main Nov 1, 2023
30 of 31 checks passed
@kkourt kkourt deleted the pr/kkourt/bpf-dual branch November 1, 2023 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

Dual-license bpf/ code as GPL 2.0 and 2-Clause BSD