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

Could not find allowed package adler #131575

Closed
Makuo12 opened this issue Oct 11, 2024 · 5 comments
Closed

Could not find allowed package adler #131575

Makuo12 opened this issue Oct 11, 2024 · 5 comments
Labels
C-discussion Category: Discussion or questions that doesn't represent real issues.

Comments

@Makuo12
Copy link

Makuo12 commented Oct 11, 2024

I tried this code:

./x tidy

I expected to see it run without any issues

Instead, this happened: tidy error: could not find allowed package adler

Meta

rustc --version --verbose:

rustc 1.81.0 (eeb90cda1 2024-09-04)
Backtrace

tidy error: could not find allowed package `adler`
Remove from PERMITTED_DEPENDENCIES list if it is no longer used.
tidy error: Dependency for library not explicitly permitted: registry+https://github.com/rust-lang/crates.io-index#[email protected]

@Makuo12 Makuo12 added the C-bug Category: This is a bug. label Oct 11, 2024
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Oct 11, 2024
@jieyouxu
Copy link
Member

jieyouxu commented Oct 11, 2024

./x tidy is not a valid subcommand. On a local checkout of latest master 01e2fff I cannot reproduce this failure. What commit are you on? Maybe try clearing the build cache and rebase on master?

I get

error: 'x.py' requires a subcommand but one was not provided
  [subcommands: build, b, check, c, clippy, fix, fmt, doc, d, test, t, miri, bench, clean, dist, install, run, r, setup, suggest, vendor, perf]

Usage: x.py <subcommand> [options] [<paths>...]

For more information, try '--help'.
Build completed unsuccessfully in 0:01:17

when I run to run ./x tidy from a fresh checkout.

@jieyouxu jieyouxu added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) S-needs-repro Status: This issue has no reproduction and needs a reproduction to make progress. S-needs-info Status: The issue lacks details necessary to triage or act on it. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. and removed C-bug Category: This is a bug. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Oct 11, 2024
@jieyouxu
Copy link
Member

Is this in the context of #131574? Those version bumps are significant, possibly breaking, as evident by changing out a major version of the adler transitive dependency underneath.

@Makuo12
Copy link
Author

Makuo12 commented Oct 12, 2024

I would roll back. Then ensure that only dependencies updates that don't affect the code are changed.

@jieyouxu
Copy link
Member

To clarify, if T-libs is okay with those version bumps, and as long as adler2 is of an allowed license, we can update the permitted dependencies allowlist.

@jieyouxu
Copy link
Member

As Trevor said in #131574 (comment), this is not a bug, this is a feature of permitted dependency checking, that we can manually update if it's permitted by relevant teams.

@jieyouxu jieyouxu closed this as not planned Won't fix, can't repro, duplicate, stale Oct 12, 2024
@jieyouxu jieyouxu added C-discussion Category: Discussion or questions that doesn't represent real issues. and removed T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) S-needs-repro Status: This issue has no reproduction and needs a reproduction to make progress. S-needs-info Status: The issue lacks details necessary to triage or act on it. labels Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-discussion Category: Discussion or questions that doesn't represent real issues.
Projects
None yet
Development

No branches or pull requests

3 participants