Fixes checksum for dependencies for Go 1.11.4+

[davividal]

Closes #7242

Quoting golang/go#29278 (comment):

Yes, pre-1.11.4, modules containing symlinks were constructed erroneously. 1.11.4 correctly omits those symlinks, which may change the sums for a few (hopefully isolated) modules.

[hashicorp-cla]

All committers have signed the CLA.

[davividal]

I tested this PR agains Go 1.11.5.

[azr]

Hey there @davividal, thanks for taking the time to do this. I need to double check a something still looks odd.

❯ go version
go version go1.11.5 darwin/amd64
❯ rm -rf ~/go/pkg/mod/cache
❯ GO111MODULE=on go mod download
...
go: finding github.com/oklog/run v1.0.0
go: verifying github.com/gotestyourself/[email protected]+incompatible: checksum mismatch
        downloaded: h1:AQwinXlbQR2HvPjQZOmDhRqsv5mZf+Jb1RnSLxcqZcI=
        go.sum:     h1:1yOKgt0XYKUg1HOKunGOSt2ocU4bxLCjmIHt0vRtVHM=
...

Edit: will try to fix it, oh dear, these downloads take forever !

[azr]

All good, thanks !!!! 🙂

[chhaj5236]

@davividal @azr I encounter same issue here with current master branch.

➜ packer git:(master) ✗ go version
go version go1.11.5 darwin/amd64

➜ packer git:(master) ✗ uname -a
...Darwin Kernel Version 17.7.0...

And it prints following errors:

go: verifying github.com/docker/[email protected]: checksum mismatch
        downloaded: h1:V/WghsgVPZl+92drOvPEwuZdHskJmrnD83V1msaV/Jc=
        go.sum:     h1:DwY2bFs8p+xf2WaQewx2hnGdjYR5K4UAaxcNPyKkTek=

It seems caused by change in #7261

[azr]

Yes, currently go modules and go vendors have a bit diverged. #7270 should sort this out.

[chhaj5236]

@azr I'm little confused here. Since I ran make dev in two different computers. One said checksum mismatch where I posted above. But another one which I tried tonight seemed working properly with the updated checksum h1:DwY2bFs8p+xf2WaQewx2hnGdjYR5K4UAaxcNPyKkTek=. Why the same code behaved differently in two environments.

[azr]

go modules are still considered experimental, it could be that checksums are different between GOOSes kubernetes/kubernetes#69040, more: golang/go#27925

[chhaj5236]

@azr Thanks for the info. That is helpful.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.