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

[20.10 backport] Makefile: enable GOPROXY to work around vanity URL being offline #685

Merged

Conversation

thaJeztah
Copy link
Member

backport of #684

The https://honnef.co/go/tools domain looks to be offline, causing projects that
do not use vendoring to fail:

go: downloading google.golang.org/protobuf v1.27.1
go: downloading github.com/matttproud/golang_protobuf_extensions v1.0.2-0.20181231171920-c182affec369
go: google.golang.org/grpc@v1.44.0 requires
google.golang.org/genproto@v0.0.0-20200526211855-cb27e3aa2013 requires
honnef.co/go/tools@v0.0.0-20190523083050-ea95bdfd59fc: unrecognized import path "honnef.co/go/tools": reading https://honnef.co/go/tools?go-get=1: 502 Bad Gateway
make: *** [Makefile:71: manpages] Error 1

For now, let's use the GOPROXY (we should look at making this a build-arg probably)

Signed-off-by: Sebastiaan van Stijn github@gone.nl
(cherry picked from commit 1dfe9a2)
Signed-off-by: Sebastiaan van Stijn github@gone.nl

The https://honnef.co/go/tools domain looks to be offline, causing projects that
do not use vendoring to fail:

    go: downloading google.golang.org/protobuf v1.27.1
    go: downloading github.com/matttproud/golang_protobuf_extensions v1.0.2-0.20181231171920-c182affec369
    go: google.golang.org/grpc@v1.44.0 requires
    google.golang.org/genproto@v0.0.0-20200526211855-cb27e3aa2013 requires
    honnef.co/go/tools@v0.0.0-20190523083050-ea95bdfd59fc: unrecognized import path "honnef.co/go/tools": reading https://honnef.co/go/tools?go-get=1: 502 Bad Gateway
    make: *** [Makefile:71: manpages] Error 1

For now, let's use the GOPROXY (we should look at making this a build-arg probably)

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 1dfe9a2)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah merged commit fea6399 into docker:20.10 May 5, 2022
@thaJeztah thaJeztah deleted the 20.10_backport_use_goproxy_by_default branch May 5, 2022 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants