Skip to content

[20.10 backport] Add workaround for CentOS 7 not working without GOPROXY#619

Merged
thaJeztah merged 1 commit into
docker:20.10from
thaJeztah:20.10_backport_fix_rootlesskit_centos
Feb 8, 2022
Merged

[20.10 backport] Add workaround for CentOS 7 not working without GOPROXY#619
thaJeztah merged 1 commit into
docker:20.10from
thaJeztah:20.10_backport_fix_rootlesskit_centos

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Jan 13, 2022

backport of #577
this is not needed (yet), but some changes may be back ported in upstream moby/moby that may cause this to be a problem, so let's prepare for that (moby/moby#42674)

This fix was previously included in the rootlesskit install script
in moby, but should've been addressed by Go 1.14 and up. Unfortunately,
this does not appear the case, and go modules without proxy is still
broken on older versions of git, such as the version of git used by
CentOS 7.

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

This fix was previously included in the rootlesskit install script
in moby, but should've been addressed by Go 1.14 and up. Unfortunately,
this does not appear the case, and go modules without proxy is still
broken on older versions of git, such as the version of git used by
CentOS 7.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 9c363de)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah merged commit d963085 into docker:20.10 Feb 8, 2022
@thaJeztah thaJeztah deleted the 20.10_backport_fix_rootlesskit_centos branch February 8, 2022 13:17
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