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

RFC 140 migration: misleading CI error for aliases #285368

Closed
zeuner opened this issue Jan 31, 2024 · 2 comments · Fixed by #285089
Closed

RFC 140 migration: misleading CI error for aliases #285368

zeuner opened this issue Jan 31, 2024 · 2 comments · Fixed by #285089
Milestone

Comments

@zeuner
Copy link
Contributor

zeuner commented Jan 31, 2024

Describe the bug

When defining an alias for a top-level package in pkgs/top-level/all-packages.nix, the RFC 140 related CI will misinterpret it as a callPackage invocation and complain. Even if aliases were forbidden by RFC 140 (which isn't obvious, the public document lists them as "unresolved"), the error message should not be misleading.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Push PR that defines an alias in pkgs/top-level/all-packages.nix
  2. See CI fail on "Check pkgs/by-name"

Expected behavior

Since there is no indication that aliases are disallowed, there should be no error at all. At the very least, it should not be misleading.

Additional context

Can be seen in #256115 (https://github.com/NixOS/nixpkgs/actions/runs/7732190624/job/21081562936?pr=256115)

Notify maintainers

Not sure where to lookup maintainers here. I guess @infinisil might be knowledgeable about this due to dedicated involvement in RFC 140 automation.

Metadata

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 4.19.0-8-amd64, Ubuntu, 20.04.6 LTS (Focal Fossa), nobuild`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.16.1`
 - nixpkgs: `/nixpkgs`

Add a 👍 reaction to issues you find important.

@infinisil
Copy link
Member

Thanks for the report! I'm already aware of this problem and created a PR to fix it yesterday: #285089

@infinisil
Copy link
Member

Actually let's leave this open until the new tooling is used in CI

@infinisil infinisil reopened this Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants