Skip to content

Tool to vet (check) Nixpkgs, including its pkgs/by-name directory

License

Notifications You must be signed in to change notification settings

NixOS/nixpkgs-vet

Repository files navigation

Nixpkgs vet (check) tool

This repository implements a program which checks Nixpkgs for many sorts of defects, both major and minor. It's named nixpkgs-vet, since it 'vets' contributions to Nixpkgs.

It implements all checks for Nixpkgs' pkgs/by-name directory as part of RFC 140.

See CONTRIBUTING.md for contributor documentation. Below is the user documentation.

Currently the only intended user for this program is Nixpkgs. Expect that the interface may be changed in breaking ways as long as Nixpkgs is adjusted to deal with it. See the pkgs/by-name Readme for how it's used in Nixpkgs.

Nix derivations

The source code contains a default.nix file, which defines a Nix function.

The function takes an attribute set with at least these attributes as its argument:

The function returns an attribute set with at least these attributes:

There is no guarantee that the derivation succeeds on systems that don't have prebuilt store paths, but it can be attempted with

nix-build https://github.com/NixOS/nixpkgs-vet/tarball/master -A build

Prebuilt store paths

The GitHub releases contain a gzip-compressed Nix Archive of the build closure of the Nix derivation with x86_64-linux as the system.

This release artifact is named x86_64-linux.nar.gz and can be imported into a local Nix store using:

storePath=$(gzip -cd x86_64-linux.nar.gz | nix-store --import | tail -1)
# To prevent it from being garbage-collected
nix-store --realise "$storePath" --add-root result

Compared to building the Nix derivations, this has the benefit that no Nix evaluation needs to take place and is therefore much faster and less storage intensive.

Binary interface

The store path acquired from the above methods contains a system-specific binary under $storePath/bin/nixpkgs-vet.

The public interface of this binary is printed by calling

result/bin/nixpkgs-vet --help

Validity checks

The following checks are performed when calling the binary:

File structure checks

  • pkgs/by-name must only contain subdirectories of the form ${shard}/${name}, called package directories.
  • The name's of package directories must be unique when lowercased.
  • name is a string only consisting of the ASCII characters a-z, A-Z, 0-9, - or _.
  • shard is the lowercased first two letters of name, expressed in Nix: shard = toLower (substring 0 2 name).
  • Each package directory must contain a package.nix file and may contain arbitrary other files.

Nix parser checks

  • Each package directory must not refer to files outside itself using symlinks or Nix path expressions.

Nix evaluation checks

Evaluate Nixpkgs with system set to x86_64-linux and check that:

  • For each package directory, the pkgs.${name} attribute must be defined as callPackage pkgs/by-name/${shard}/${name}/package.nix args for some args.
  • For each package directory, pkgs.lib.isDerivation pkgs.${name} must be true.

Ratchet checks

Furthermore, this tool implements certain ratchet checks. This allows gradually phasing out deprecated patterns without breaking the base branch or having to migrate it all at once. It works by not allowing new instances of the pattern to be introduced, but allowing already existing instances. The existing instances are coming from <BASE_NIXPKGS>, which is then checked against <NIXPKGS> for new instances. Ratchets should be removed eventually once the pattern is not used anymore.

The current ratchets are:

  • New manual definitions of pkgs.${name} (e.g. in pkgs/top-level/all-packages.nix) with args = { } (see nix evaluation checks) must not be introduced.
  • New top-level packages defined using pkgs.callPackage must be defined with a package directory.
    • Once a top-level package uses pkgs/by-name, it also can't be moved back out of it.