Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/go: go mod tidy revoke the modify about go.sum made by go build #29163

Closed
Sherlock-Holo opened this issue Dec 9, 2018 · 2 comments
Closed
Labels
FrozenDueToAge modules NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone

Comments

@Sherlock-Holo
Copy link

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

go version go1.11.2 linux/amd64

Does this issue reproduce with the latest release?

Yes

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

go env Output
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/sherlock/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/sherlock/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/sherlock/git/recruitment/interview/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build973697242=/tmp/go-build -gno-record-gcc-switches"

What did you do?

go mod tidy and then git add . and then go build -v , go mod tidy again

What did you expect to see?

go.sum isn't modified

What did you see instead?

use git diff and found go.sum has been modified. go mod tidy again and it revoke the modify which go build -v do

@bcmills bcmills added WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. modules labels Dec 10, 2018
@bcmills
Copy link
Contributor

bcmills commented Dec 10, 2018

If it involves a replaced module, this is a duplicate of #27868.

Otherwise, it needs more detail: please provide an exact sequence of commands we can execute to reproduce the problem, including the directory structure and the contents of the .go and go.mod files.

@bcmills bcmills added this to the Go1.13 milestone Dec 10, 2018
@thepudds thepudds changed the title go mod tidy revoke the modify about go.sum made by go build cmd/go: go mod tidy revoke the modify about go.sum made by go build Dec 10, 2018
@bcmills
Copy link
Contributor

bcmills commented Dec 12, 2018

Provisionally closing as a duplicate of #27868. Please let us know if you can reproduce this without replacement modules.

@bcmills bcmills closed this as completed Dec 12, 2018
@golang golang locked and limited conversation to collaborators Dec 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge modules NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Development

No branches or pull requests

3 participants