Opened 4 years ago

Closed 4 years ago

#61182 closed defect (fixed)

golang portgroup sets default worksrcdir incorrectly

Reported by: jmroot (Joshua Root) Owned by: Aaron Madlon-Kay <amake@…>
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: amake (Aaron Madlon-Kay)
Port:

Description

set gopath              ${workpath}/gopath
default worksrcdir      {${gopath}/src/${go.package}}

This is wrong because worksrcdir is supposed to be a relative path. The corresponding full path, constructed from workpath and worksrcdir, is is worksrcpath.

Change History (5)

comment:1 Changed 4 years ago by amake (Aaron Madlon-Kay)

Thanks, it's weird that it was working anyway.

I've fixed this locally but I want to test all the existing golang ports before I push.

Memo: all ports using golang (currently 85) can be found with

find . -name Portfile | xargs grep -l -E 'PortGroup +golang' | xargs -n 1 dirname | xargs -n 1 basename

comment:2 Changed 4 years ago by jmroot (Joshua Root)

Thanks. Yeah, I wasn't sure if anything was relying on this or I would have just fixed it myself.

comment:3 Changed 4 years ago by amake (Aaron Madlon-Kay)

I think it was just an oversight; the correction is to simply make ${gopath} into a literal: gopath

I had two build failures: istioctl and kubergrunt. I will investigate them and make the fix if it's OK.

comment:4 Changed 4 years ago by amake (Aaron Madlon-Kay)

The failures were unrelated; reported as #61184 and #61185

comment:5 Changed 4 years ago by Aaron Madlon-Kay <amake@…>

Owner: set to Aaron Madlon-Kay <amake@…>
Resolution: fixed
Status: newclosed

In da7eb4eb859c0e9a22596d2cacabd6626af51852/macports-ports (master):

golang portgroup: fix worksrcdir

${worksrcdir} should be relative to ${workpath}

Closes #61182

Note: See TracTickets for help on using tickets.