Page MenuHomeFreeBSD
Differential D58005

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

Authored by des on Thu, Jul 2, 10:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jul 9, 10:22 PM
Unknown Object (File)
Thu, Jul 9, 5:32 PM
Unknown Object (File)
Thu, Jul 9, 10:54 AM
Unknown Object (File)
Thu, Jul 9, 2:33 AM
Unknown Object (File)
Thu, Jul 9, 2:12 AM
Unknown Object (File)
Wed, Jul 8, 2:56 PM
Unknown Object (File)
Tue, Jul 7, 8:26 PM
Unknown Object (File)
Tue, Jul 7, 6:43 PM
View All 16 Files
Subscribers
imp
markj
ziaee

Details

Reviewers
markj
Group Reviewers
srcmgr
Commits
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 Not Applicable
Unit
Tests Not Applicable

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
markj added a subscriber: markj.Thu, Jul 2, 1:26 PM

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?

des updated this revision to Diff 181220.Thu, Jul 2, 3:06 PM

rf

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

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.

Revision Contents
Changeset List

PathSize
share/
man/
man5/
tools/
build/
options/
17 lines

Diff 181221

View Options

share/man/man5/src.conf.5

Loading...
View Options

tools/build/options/WITH_CCACHE_BUILD

Loading...