NixOS/nix

nix path-info --json always return with exit code 0 contradicting docs

Open

#10.886 aperta il 10 giu 2024

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (1293 fork)batch import
buggood first issuenew-cli

Metriche repository

Star
 (9803 star)
Metriche merge PR
 (Merge medio 4g 12h) (86 PR mergiate in 30 g)

Descrizione

Describe the bug

From the docs:

          Warning

         Note that nix path-info does not build or substitute the installables you specify. Thus, if the corresponding store paths don’t already exist, this command will
        fail. You can use nix build to ensure that they exist.

Steps To Reproduce

nix path-info .#derivation-didnt-build

returns

error: path '/nix/store/qd2prbj1wjbf0r816s60ps15i7hw5l90-nixos-system-unnamed-24.05.20240530.d24e7fd' is not valid
$ echo $?
1

However

nix path-info .#derivation-didnt-build --json

returns

{"/nix/store/qd2prbj1wjbf0r816s60ps15i7hw5l90-nixos-system-unnamed-24.05.20240530.d24e7fd":null}% 
$ echo $?
0

Expected behavior

nix path-info --json

should return non-zero exit code if derivation isnt built

nix-env --version output

Additional context

I'm using this to check if a derivation is in cache but this command silently broke broke as I added --json

if ! nix path-info  --eval-store auto --store s3://mybucket ; then
  nix copy --eval-store --auto --to s3://mybucket  
fi

Priorities

Add :+1: to issues you find important.

Guida contributor

Direzione di ricerca
Il bug si trova nella gestione dell'output JSON del comando `path info`. Esamina il codice sorgente in `src/libcmd/` o `src/nix/` per l'implementazione di `path info`. In particolare, osserva come viene stampato il formato JSON e come vengono impostati i codici di uscita dopo l'elaborazione. La versione non JSON restituisce correttamente il codice di uscita 1 quando i percorsi non sono validi, quindi una verifica simile manca nel ramo JSON. Probabilmente, il risultato di `Store::queryPathInfo` non viene controllato per validità prima di stampare il JSON. Un buon punto di partenza è confrontare i percorsi di codice per l'output regolare e JSON nella stessa funzione.
Tech stack
cppshell
Dominio
clibackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
understanding of Nix CLIC++ basicsexit code handling
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40