Page MenuHomeFreeBSD

Go ports: cleanup, finish transition to USES=go started with r505321 (cat. misc-shells)
ClosedPublic

Authored by dmgk on Sep 21 2019, 9:16 PM.
Tags
None
Referenced Files
F110706249: D21743.id62727.diff
Sat, Feb 22, 3:13 AM
F110667577: D21743.id62402.diff
Fri, Feb 21, 4:39 PM
Unknown Object (File)
Fri, Feb 21, 11:20 AM
Unknown Object (File)
Sun, Feb 9, 5:15 AM
Unknown Object (File)
Sat, Feb 8, 8:58 AM
Unknown Object (File)
Jan 22 2025, 7:05 AM
Unknown Object (File)
Jan 17 2025, 6:10 PM
Unknown Object (File)
Dec 28 2024, 8:14 AM
Subscribers

Details

Summary

Go ports: cleanup, finish transition to USES=go started with r505321 (cat. misc-shells)

  • Remove custom build/install targets left in place after r505321
  • Switch to the new GO_TARGET tuple syntax introduced in r512001
  • Switch to go:modules when upstream already uses them
Test Plan

poudriere bulk -C: OK (113i, 120a)
exp-run

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tobik added inline comments.
net-mgmt/kapacitor/Makefile
55–57 ↗(On Diff #62402)

This is in the wrong place. Consider moving this upwards to after GH_SUBDIR.

net/traefik/Makefile
32–36 ↗(On Diff #62402)

Consider moving this to after USE_RC_SUBR too.

security/meek/Makefile
41–45 ↗(On Diff #62402)

go test support in USES=go next? 0:-)

This revision is now accepted and ready to land.Sep 23 2019, 4:39 AM
dmgk marked 2 inline comments as done.

Fix variables ordering.

This revision now requires review to proceed.Sep 23 2019, 12:09 PM
This revision is now accepted and ready to land.Sep 23 2019, 4:00 PM
security/meek/Makefile
41–45 ↗(On Diff #62402)

Something like GO_TEST_TARGET? I'll look into it..

LGTM, same thing as the other reviews, wait for the exp-run and also for the portmgr approval.