golang/go

cmd/go: several commands require the build cache but don't use it

Aperta

#39.882 aperta il 26 giu 2020

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
GoCommandNeedsFixhelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Steps to repro:

docker run -i golang:1.15beta1 <<-SCRIPT

        set -ex

        go version
        go env

        mkdir /tmp/foo
        cd /tmp/foo
        go mod init test
        go mod edit -go=1.13

        HOME= GOCACHE= go mod edit -go=1.14

SCRIPT

I think this should succeed. go mod edit -go=X only has to modify a line in go.mod, it shouldn't require the build cache to exist or be writeable. However, that last command fails:

+ go mod edit -go=1.14
build cache is required, but could not be located: GOCACHE is not defined and neither $XDG_CACHE_HOME nor $HOME are defined

I encountered this while writing some tests, which are run in a temporary directory with a near-empty environment (partly to not pollute the user's $HOME and such).

Guida contributor

Direzione di ricerca
Esamina il codice sorgente Go di 'go mod edit' (src/cmd/go/internal/modcmd/edit.go) e traccia il percorso in cui viene verificata la cache di build. Identifica dove il requisito della cache viene imposto inutilmente e proponi una modifica per saltare tale controllo per i comandi che non ne hanno bisogno.
Tech stack
go
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Gounderstanding of build cache mechanics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60