Skip to content

Commit 5da57bc

Browse files
committed
pkg/gcp/build: Pass build options by pointer (hugeParam)
Signed-off-by: Stephen Augustus <[email protected]>
1 parent e790be2 commit 5da57bc

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

cmd/gcbuilder/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ func main() {
9090
log.Fatalf("Failed to chdir to build directory (%s): %v", o.BuildDir, err)
9191
}
9292

93-
errors := build.RunBuildJobs(o)
93+
errors := build.RunBuildJobs(&o)
9494
if len(errors) != 0 {
9595
log.Fatalf("Failed to run some build jobs: %v", errors)
9696
}

pkg/gcp/build/build.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ func (o *Options) uploadBuildDir(targetBucket string) (string, error) {
111111
return uploaded, nil
112112
}
113113

114-
func getExtraSubs(o Options) map[string]string {
114+
func getExtraSubs(o *Options) map[string]string {
115115
envs := strings.Split(o.EnvPassthrough, ",")
116116
subs := map[string]string{}
117117
for _, e := range envs {
@@ -123,7 +123,7 @@ func getExtraSubs(o Options) map[string]string {
123123
return subs
124124
}
125125

126-
func runSingleJob(o Options, jobName, uploaded, version string, subs map[string]string) error {
126+
func runSingleJob(o *Options, jobName, uploaded, version string, subs map[string]string) error {
127127
s := make([]string, 0, len(subs)+1)
128128
for k, v := range subs {
129129
s = append(s, fmt.Sprintf("_%s=%s", k, v))
@@ -189,7 +189,7 @@ func runSingleJob(o Options, jobName, uploaded, version string, subs map[string]
189189

190190
type variants map[string]map[string]string
191191

192-
func getVariants(o Options) (variants, error) {
192+
func getVariants(o *Options) (variants, error) {
193193
content, err := ioutil.ReadFile(path.Join(o.ConfigDir, "variants.yaml"))
194194
if err != nil {
195195
if !os.IsNotExist(err) {
@@ -216,7 +216,7 @@ func getVariants(o Options) (variants, error) {
216216
return v.Variants, nil
217217
}
218218

219-
func RunBuildJobs(o Options) []error {
219+
func RunBuildJobs(o *Options) []error {
220220
var uploaded string
221221
if o.ScratchBucket != "" {
222222
if !o.NoSource {

0 commit comments

Comments
 (0)