search

rust-fuzz / libfuzzer

Rust bindings and utilities for LLVM’s libFuzzer
Apache License 2.0
206 stars 44 forks source link

Add license files #75

Closed rcvalle closed 3 years ago

nagisa commented 3 years ago

LGTM, but should have another pair of eyes on it.

(Context: the Apache/MIT license is the same as for cargo-fuzz, libfuzzer's license field in Cargo.toml also specifies NSCA, which used to apply to the code taken from the LLVM project, before it changed license to Apache-2.0-with-LLVM-exception OR NSCA)

Manishearth commented 3 years ago

Can we stick the relevant license file in the libfuzzer folder and have a LICENSE file that says all this?

nagisa commented 3 years ago

@Manishearth that's what this PR does – the libfuzzer folder contains the license files copied from the compiler_rt, and the top level contains just the APACHE/MIT license files, or did you mean something else?

Manishearth commented 3 years ago

Oh, I missed that. This seems fine.

Would still kind of be nice for the license situation to be explained in a separate file.

rcvalle commented 3 years ago

Thank you, @nagisa and @Manishearth, for your time!