Page MenuHomeFreeBSD

Mk/Uses/go.mk: Better ARM support
ClosedPublic

Authored by dg_syrec.org on Apr 12 2019, 8:15 PM.

Details

Summary

This is a companion patch for https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221297

  • If not set explicitly by the port, set CGO_ENABLED=1. CGO_ENABLED already defaults to 1 on all supported archs except ARM [1], so this change effectively just enables it on ARM.
  • If not set explicitly by the port, set GOARM from ARCH for ARM archs.

[1] https://github.com/golang/go/blob/master/src/cmd/dist/build.go#L1486

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

dg_syrec.org created this revision.Apr 12 2019, 8:15 PM
jlaffaye accepted this revision.Apr 13 2019, 9:27 AM

LGTM, but do we know the reason why CGO is not enabled on arm by default by upstream ?

LGTM, but do we know the reason why CGO is not enabled on arm by default by upstream ?

I asked in the PR - https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221297#c27. I also think it is just an upstream oversight at this point.

This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.