tidy: add license exceptions for rustc-perf
`ring` is included because it is an optional dependency of `hyper`, which is a training data in rustc-pref for optimized build. The license of it is generally `ISC AND MIT AND OpenSSL`, though the `package.license` field is not set. See https://github.com/briansmith/ring/issues/902 `git+https://github.com/rust-lang/team` git source is from `rust_team_data`, which is used by `site` in src/tools/rustc-perf. This doesn't really a part of the compiler, so doesn't really affect the bootstrapping process. See https://github.com/rust-lang/rustc-perf/pull/1914.
This commit is contained in:
parent
e24be071e3
commit
377870386f
2 changed files with 41 additions and 1 deletions
|
@ -67,6 +67,7 @@ pub(crate) const WORKSPACES: &[(&str, ExceptionList, Option<(&[&str], &[&str])>)
|
||||||
//("src/tools/miri/test-cargo-miri", &[], None), // FIXME uncomment once all deps are vendored
|
//("src/tools/miri/test-cargo-miri", &[], None), // FIXME uncomment once all deps are vendored
|
||||||
//("src/tools/miri/test_dependencies", &[], None), // FIXME uncomment once all deps are vendored
|
//("src/tools/miri/test_dependencies", &[], None), // FIXME uncomment once all deps are vendored
|
||||||
("src/tools/rust-analyzer", EXCEPTIONS_RUST_ANALYZER, None),
|
("src/tools/rust-analyzer", EXCEPTIONS_RUST_ANALYZER, None),
|
||||||
|
("src/tools/rustc-perf", EXCEPTIONS_RUSTC_PERF, None),
|
||||||
("src/tools/x", &[], None),
|
("src/tools/x", &[], None),
|
||||||
// tidy-alphabetical-end
|
// tidy-alphabetical-end
|
||||||
];
|
];
|
||||||
|
@ -142,6 +143,22 @@ const EXCEPTIONS_RUST_ANALYZER: ExceptionList = &[
|
||||||
// tidy-alphabetical-end
|
// tidy-alphabetical-end
|
||||||
];
|
];
|
||||||
|
|
||||||
|
const EXCEPTIONS_RUSTC_PERF: ExceptionList = &[
|
||||||
|
// tidy-alphabetical-start
|
||||||
|
("alloc-no-stdlib", "BSD-3-Clause"),
|
||||||
|
("alloc-stdlib", "BSD-3-Clause"),
|
||||||
|
("brotli", "BSD-3-Clause/MIT"),
|
||||||
|
("brotli-decompressor", "BSD-3-Clause/MIT"),
|
||||||
|
("encoding_rs", "(Apache-2.0 OR MIT) AND BSD-3-Clause"),
|
||||||
|
("inferno", "CDDL-1.0"),
|
||||||
|
("instant", "BSD-3-Clause"),
|
||||||
|
("ring", NON_STANDARD_LICENSE), // see EXCEPTIONS_NON_STANDARD_LICENSE_DEPS for more.
|
||||||
|
("ryu", "Apache-2.0 OR BSL-1.0"),
|
||||||
|
("snap", "BSD-3-Clause"),
|
||||||
|
("subtle", "BSD-3-Clause"),
|
||||||
|
// tidy-alphabetical-end
|
||||||
|
];
|
||||||
|
|
||||||
const EXCEPTIONS_CRANELIFT: ExceptionList = &[
|
const EXCEPTIONS_CRANELIFT: ExceptionList = &[
|
||||||
// tidy-alphabetical-start
|
// tidy-alphabetical-start
|
||||||
("cranelift-bforest", "Apache-2.0 WITH LLVM-exception"),
|
("cranelift-bforest", "Apache-2.0 WITH LLVM-exception"),
|
||||||
|
@ -178,6 +195,20 @@ const EXCEPTIONS_UEFI_QEMU_TEST: ExceptionList = &[
|
||||||
("r-efi", "MIT OR Apache-2.0 OR LGPL-2.1-or-later"), // LGPL is not acceptible, but we use it under MIT OR Apache-2.0
|
("r-efi", "MIT OR Apache-2.0 OR LGPL-2.1-or-later"), // LGPL is not acceptible, but we use it under MIT OR Apache-2.0
|
||||||
];
|
];
|
||||||
|
|
||||||
|
/// Placeholder for non-standard license file.
|
||||||
|
const NON_STANDARD_LICENSE: &str = "NON_STANDARD_LICENSE";
|
||||||
|
|
||||||
|
/// These dependencies have non-standard licenses but are genenrally permitted.
|
||||||
|
const EXCEPTIONS_NON_STANDARD_LICENSE_DEPS: &[&str] = &[
|
||||||
|
// `ring` is included because it is an optional dependency of `hyper`,
|
||||||
|
// which is a training data in rustc-perf for optimized build.
|
||||||
|
// The license of it is generally `ISC AND MIT AND OpenSSL`,
|
||||||
|
// though the `package.license` field is not set.
|
||||||
|
//
|
||||||
|
// See https://github.com/briansmith/ring/issues/902
|
||||||
|
"ring",
|
||||||
|
];
|
||||||
|
|
||||||
/// These are the root crates that are part of the runtime. The licenses for
|
/// These are the root crates that are part of the runtime. The licenses for
|
||||||
/// these and all their dependencies *must not* be in the exception list.
|
/// these and all their dependencies *must not* be in the exception list.
|
||||||
const RUNTIME_CRATES: &[&str] = &["std", "core", "alloc", "test", "panic_abort", "panic_unwind"];
|
const RUNTIME_CRATES: &[&str] = &["std", "core", "alloc", "test", "panic_abort", "panic_unwind"];
|
||||||
|
@ -610,6 +641,11 @@ fn check_license_exceptions(metadata: &Metadata, exceptions: &[(&str, &str)], ba
|
||||||
for pkg in metadata.packages.iter().filter(|p| p.name == *name) {
|
for pkg in metadata.packages.iter().filter(|p| p.name == *name) {
|
||||||
match &pkg.license {
|
match &pkg.license {
|
||||||
None => {
|
None => {
|
||||||
|
if *license == NON_STANDARD_LICENSE
|
||||||
|
&& EXCEPTIONS_NON_STANDARD_LICENSE_DEPS.contains(&pkg.name.as_str())
|
||||||
|
{
|
||||||
|
continue;
|
||||||
|
}
|
||||||
tidy_error!(
|
tidy_error!(
|
||||||
bad,
|
bad,
|
||||||
"dependency exception `{}` does not declare a license expression",
|
"dependency exception `{}` does not declare a license expression",
|
||||||
|
|
|
@ -4,7 +4,11 @@ use std::fs;
|
||||||
use std::path::Path;
|
use std::path::Path;
|
||||||
|
|
||||||
/// List of allowed sources for packages.
|
/// List of allowed sources for packages.
|
||||||
const ALLOWED_SOURCES: &[&str] = &["\"registry+https://github.com/rust-lang/crates.io-index\""];
|
const ALLOWED_SOURCES: &[&str] = &[
|
||||||
|
r#""registry+https://github.com/rust-lang/crates.io-index""#,
|
||||||
|
// This is `rust_team_data` used by `site` in src/tools/rustc-perf,
|
||||||
|
r#""git+https://github.com/rust-lang/team#a5260e76d3aa894c64c56e6ddc8545b9a98043ec""#,
|
||||||
|
];
|
||||||
|
|
||||||
/// Checks for external package sources. `root` is the path to the directory that contains the
|
/// Checks for external package sources. `root` is the path to the directory that contains the
|
||||||
/// workspace `Cargo.toml`.
|
/// workspace `Cargo.toml`.
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue