NixOS/nix

builtins.fetchTree behaves differently if uri or attrs are parsed

Open

#12 860 ouverte le 1 avr. 2025

Voir sur GitHub
 (3 commentaires) (1 réaction) (0 assignés)C++ (1 293 forks)batch import
bugfetchinggood first issue

Métriques du dépôt

Stars
 (9 803 stars)
Métriques de merge PR
 (Merge moyen 4j 12h) (86 PRs mergées en 30 j)

Description

While implementing builtins.fetchTree for https://snix.dev/ I've noticed revCount is missing when it shouldn't:

nixpkgs on  master [$] took 3s 
❯ /nix/store/pkw2fif96sahwnnkfxsmdzs0dl1nr5ln-nix-2.26.3/bin/nix-instantiate -E "builtins.fetchTree git+https://github.com/cachix/devenv" --eval
{ lastModified = 1743521698; lastModifiedDate = "20250401153458"; narHash = "sha256-bvl7YmNZjGEZhyH86mGiJdhZGMaRot1TMM6cjwrwH6k="; outPath = "/nix/store/b9319a8ffx2h1vd1bv7ip2ris0sf2sq2-source"; rev = "4917f16d045f12d938d0a29fb1cbfb9e24ae6d0d"; revCount = 3463; shortRev = "4917f16"; submodules = false; }

nixpkgs on  master [$] 
❯ /nix/store/pkw2fif96sahwnnkfxsmdzs0dl1nr5ln-nix-2.26.3/bin/nix-instantiate -E "builtins.fetchTree (builtins.parseFlakeRef git+https://github.com/cachix/devenv)" --eval
{ lastModified = 1743521698; lastModifiedDate = "20250401153458"; narHash = "sha256-bvl7YmNZjGEZhyH86mGiJdhZGMaRot1TMM6cjwrwH6k="; outPath = "/nix/store/b9319a8ffx2h1vd1bv7ip2ris0sf2sq2-source"; rev = "4917f16d045f12d938d0a29fb1cbfb9e24ae6d0d"; shortRev = "4917f16"; submodules = false; }

Property builtins.fetchTree input == builtins.fetchTree (builtins.parseFlakeRef input) should hold.

Guide contributeur

Direction de recherche
Étudiez l'écart entre fetchTree lorsqu'il est appelé directement avec une chaîne URI par rapport à lorsqu'il est appelé avec le résultat de builtins.parseFlakeRef. Comparez les chemins de code dans la source de Nix, probablement dans src/libexpr/primops/fetchTree.cc et le code de gestion des flake associé. Recherchez des différences dans la façon dont revCount est calculé ou omis dans le chemin attrs.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Nix builtinsfetchTreeflake referencesC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40