src.conf(5): Suggest using ccache4
ClosedPublic
Actions

Authored by des on Thu, Jul 2, 10:29 AM.

Details

Reviewers

Group Reviewers

Commits

rGe16857e08fea: src.conf(5): Suggest using ccache4
rGb775479ca2c4: src.conf(5): Suggest using ccache4
rGa2545f68f64b: src.conf(5): Suggest using ccache4

Summary

We have separate ports for Ccache 3 and 4. Suggest both, rather than
only the Ccache 3 port. Rearrange the text somewhat to avoid an
excessively ragged edge on a standard 80-column terminal.

MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 74479
Build 71362: arc lint + arc unit

Event Timeline

des created this revision.Thu, Jul 2, 10:29 AM

Herald added subscribers: ziaee, imp. · View Herald TranscriptThu, Jul 2, 10:29 AM

des requested review of this revision.Thu, Jul 2, 10:29 AM

Harbormaster completed remote builds in B74478: Diff 181212.Thu, Jul 2, 10:29 AM

The content looks ok, but the diff should be modifying tools/build/options/WITH_CCACHE_BUILD.

share/man/man5/src.conf.5
344	Are all of these variables supposed to be set in the environment, or in /etc/src.conf?

Harbormaster completed remote builds in B74479: Diff 181220.Thu, Jul 2, 3:06 PM

markj accepted this revision as: markj.Thu, Jul 2, 3:14 PM

This revision is now accepted and ready to land.Thu, Jul 2, 3:14 PM

Closed by commit rGa2545f68f64b: src.conf(5): Suggest using ccache4 (authored by des). · Explain WhyThu, Jul 2, 3:17 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGa2545f68f64b: src.conf(5): Suggest using ccache4.

ziaee added inline comments.Thu, Jul 2, 6:29 PM

share/man/man5/src.conf.5
335–336	I think we should not say "port or package" because of my experience messing this up in intro.1 and observing user confusion. There is really only installing the package, the package can be downloaded pre-made or you can build the package yourself with the ports framework, but either way the package is installed. Also all the packages are software from the ports collection.