Page MenuHomeFreeBSD

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

Authored by dmgk on Sep 21 2019, 7:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 8, 2:50 PM
Unknown Object (File)
Tue, Oct 8, 2:49 PM
Unknown Object (File)
Tue, Oct 8, 2:48 PM
Unknown Object (File)
Tue, Oct 8, 2:47 PM
Unknown Object (File)
Tue, Oct 8, 2:47 PM
Unknown Object (File)
Oct 4 2024, 6:44 PM
Unknown Object (File)
Sep 27 2024, 6:52 AM
Unknown Object (File)
Sep 22 2024, 11:05 PM
Subscribers

Details

Summary

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

  • 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

audio/pms-devel/Makefile
21 ↗(On Diff #62399)

Many dependencies have changed here (some tags do not match the commits from before). Why? Make sure to bump PORTREVISION to force a rebuild for this.

dmgk added inline comments.
audio/pms-devel/Makefile
21 ↗(On Diff #62399)

Hmm.. I'm not sure where original dependencies came from, upstream doesn't seem to have used anything except modules. The new GH_TUPLE was generated by gomod-vendor target. But this definitely needs a PORTREVISION bump, thanks for spotting.

dmgk marked an inline comment as done.

Bump audio/pms-devel PORTREVISION for dependencies change.

tobik requested changes to this revision.Sep 23 2019, 3:42 PM
tobik added inline comments.
editors/micro/Makefile
46 ↗(On Diff #62460)

Typo here. Should be GO_BUIILDFLAGS.

This revision now requires changes to proceed.Sep 23 2019, 3:42 PM

Fix GO_BUIILDFLAGS typo in editors/micro/Makefile

dmgk marked an inline comment as done.Sep 23 2019, 3:48 PM
This revision is now accepted and ready to land.Sep 23 2019, 4:00 PM

It looks good to me, but you need to wait for the exp-run and also portmgr approval.
Thanks to work on this.