Page MenuHomeFreeBSD

Makefile: re-wordsmith the blurb about xtoolchain ports
ClosedPublic

Authored by kevans on Oct 16 2020, 5:47 PM.
Tags
None
Referenced Files
F132361959: D26820.id78326.diff
Thu, Oct 16, 5:52 AM
F132358218: D26820.id78326.diff
Thu, Oct 16, 5:13 AM
F132358209: D26820.id78324.diff
Thu, Oct 16, 5:13 AM
F132358205: D26820.id.diff
Thu, Oct 16, 5:13 AM
F132358203: D26820.id79551.diff
Thu, Oct 16, 5:13 AM
F132307368: D26820.diff
Wed, Oct 15, 6:06 PM
Unknown Object (File)
Tue, Oct 14, 4:05 AM
Unknown Object (File)
Mon, Oct 13, 5:00 PM
Subscribers

Details

Summary

The new version only includes a specific version once, and uses that that's currently advised by tinderbox: -gcc6.

It also advises just installing the pkg, but mentions in a side-note at the end where to find the source in the ports tree.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Seems good to me, though maybe N would be better than X to avoid confusion with the X in xtoolchain leading people to think gccX means a cross GCC if they don't bother looking too closely at the example? That or change "xtoolchain" to "cross toolchain" ("cross-toolchain"?), which also nicely matches CROSS_TOOLCHAIN.

Seems good to me, though maybe N would be better than X to avoid confusion with the X in xtoolchain leading people to think gccX means a cross GCC if they don't bother looking too closely at the example? That or change "xtoolchain" to "cross toolchain" ("cross-toolchain"?), which also nicely matches CROSS_TOOLCHAIN.

I'm tempted to do both, in case someone tries to make the logical leap from cross -> X anyways. Writing 'xtoolchain' was kind of just habit from the old ports that should be retired, and we probably shouldn't accidentally draw someone towards those ones.

+1 to the X -> N and xtoolchain -> "cross toolchain" changes.

Makefile
100 โ†—(On Diff #78324)
kevans marked an inline comment as done.

Incorporate feedback

One minor change but otherwise ๐Ÿ‘.

Makefile
93 โ†—(On Diff #78326)
This revision is now accepted and ready to land.Oct 16 2020, 6:27 PM
Makefile
93 โ†—(On Diff #78326)

I will fix this one pre-commit, thanks :)