Page MenuHomeFreeBSD

Implement build and test using mage
Needs RevisionPublic

Authored by otis on Jul 29 2020, 3:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 9, 3:14 PM
Unknown Object (File)
Wed, Nov 19, 8:56 AM
Unknown Object (File)
Tue, Nov 18, 3:15 PM
Unknown Object (File)
Tue, Nov 18, 2:26 PM
Unknown Object (File)
Tue, Nov 18, 7:15 AM
Unknown Object (File)
Sun, Nov 16, 3:42 AM
Unknown Object (File)
Sun, Nov 16, 1:29 AM
Unknown Object (File)
Nov 10 2025, 5:00 AM

Details

Reviewers
dmgk
Group Reviewers
portmgr
Summary

While working on new sysutils/beats7, I found out that it is now being built using devel/mage.

This proposed change implements new argument for USES=go (mage)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

otis requested review of this revision.Jul 29 2020, 3:41 PM

Could you use devel/arcanist, or at least generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.

Mk/Uses/go.mk
143–150

In USES, we try to follow the order of stuff in a port's Makefile, so variable definitions should happen before targets start to be defined, so, just before the post-extract target.

180–185

Same for here.

  • diff with full context
  • reordered statements
mat added inline comments.
Mk/Uses/go.mk
138–145

We usually write VARIABLE=<tab> to let code breathe, it makes it more readable for the brain by pre-splitting logical blocks of text.

141–143

You should probably put all this one one line, it would probably easier to read.

dmgk requested changes to this revision.Aug 3 2020, 1:41 PM

I think the decision was to put this on hold unless more Go ports use mage for build.

This revision now requires changes to proceed.Aug 3 2020, 1:41 PM