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
Unknown Object (File)
Sun, Apr 21, 12:35 PM
Unknown Object (File)
Sun, Apr 21, 12:35 PM
Unknown Object (File)
Sun, Apr 21, 12:35 PM
Unknown Object (File)
Sun, Apr 21, 11:56 AM
Unknown Object (File)
Sun, Apr 21, 11:54 AM
Unknown Object (File)
Mar 9 2024, 10:03 AM
Unknown Object (File)
Mar 9 2024, 10:03 AM
Unknown Object (File)
Mar 9 2024, 10:03 AM
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 Skipped
Unit
Tests Skipped
Build Status
Buildable 34217

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
kevans marked an inline comment as done.

Incorporate feedback

One minor change but otherwise 👍.

Makefile
93
This revision is now accepted and ready to land.Oct 16 2020, 6:27 PM
Makefile
93

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