Gentoo Archives: gentoo-dev

From: William Hubbs <williamh@g.o>
To: gentoo-dev@l.g.o
Cc: robbat2@g.o, zmedico@g.o, William Hubbs <williamh@g.o>
Subject: [gentoo-dev] [PATCH 1/2] go-module.eclass: fix GOPROXY export
Date: Fri, 21 May 2021 15:46:40
Message-Id: 20210521154559.11079-2-williamh@gentoo.org
In Reply to: [gentoo-dev] [PATCH 0/2] go-module.eclass cleanups by William Hubbs
1 This variable should be exported in the go-module_set_globals function
2 since it is not needed unless EGO_SUM is used.
3
4 Signed-off-by: William Hubbs <williamh@g.o>
5 ---
6 eclass/go-module.eclass | 6 ++++--
7 1 file changed, 4 insertions(+), 2 deletions(-)
8
9 diff --git a/eclass/go-module.eclass b/eclass/go-module.eclass
10 index c9a7ab12eaf..9d64ad48b43 100644
11 --- a/eclass/go-module.eclass
12 +++ b/eclass/go-module.eclass
13 @@ -232,6 +232,9 @@ go-module_set_globals() {
14 readonly EGO_SUM_SRC_URI
15 readonly _GOMODULE_GOSUM_REVERSE_MAP
16
17 + # export the GOPROXY setting
18 + export GOPROXY="file://${T}/go-proxy"
19 +
20 # Set the guard that we are safe
21 _GO_MODULE_SET_GLOBALS_CALLED=1
22 }
23 @@ -268,7 +271,7 @@ _go-module_src_unpack_gosum() {
24 die "go-module_set_globals must be called in global scope"
25 fi
26
27 - local goproxy_dir="${T}/go-proxy"
28 + local goproxy_dir="${GOPROXY/file:\/\//}"
29 mkdir -p "${goproxy_dir}" || die
30
31 # For each Golang module distfile, look up where it's supposed to go, and
32 @@ -293,7 +296,6 @@ _go-module_src_unpack_gosum() {
33 unpack "$f"
34 fi
35 done
36 - export GOPROXY="file://${goproxy_dir}"
37
38 # Validate the gosum now
39 _go-module_src_unpack_verify_gosum
40 --
41 2.26.3