golang/go

x/tools/cmd/goimports: do not prefix packages from GOROOT if it is inside a module

Open

#31.206 aperta il 2 apr 2019

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
NeedsFixToolshelp wanted

Metriche repository

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

Descrizione

What version of Go are you using (go version)?

Does this issue reproduce with the latest release?

Yes.

What operating system and processor architecture are you using (go env)?

What did you do?

I have a GOROOT inside a module (a per-repository Go installation for a hermetic build).

I run goimports, it tries to resolve a package (say, reflect), it loads packages from GOROOT, and they end up being prefixed with a path inside the current module.

What did you expect to see?

package mypkg

import "reflect"

type x reflect.Type

What did you see instead?

package mypkg

import "my.module/_deps/go-1.12beta2/src/reflect"

type x reflect.Type

How to fix

[Patch redacted by @bcmills. Please send a PR or CL so that we can verify CLA compliance.]

Guida contributor

Direzione di ricerca
Indaga come goimports risolve i percorsi dei pacchetti quando GOROOT è all'interno di un modulo. Esamina il codice in x/tools/goimports che determina se anteporre il percorso del modulo alle importazioni. Riproduci con una build hermetic di Go all'interno di un modulo e traccia la logica.
Tech stack
go
Dominio
backendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Gounderstanding of Go modules
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60