Page MenuHomeFreeBSD
Differential D33716

Add clarifying comments
ClosedPublic
Actions

Authored by imp on Jan 1 2022, 11:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 1, 11:15 PM
Unknown Object (File)
Mon, Dec 30, 10:48 AM
Unknown Object (File)
Dec 10 2024, 6:19 PM
Unknown Object (File)
Nov 5 2024, 11:51 AM
Unknown Object (File)
Oct 30 2024, 5:17 AM
Unknown Object (File)
Oct 23 2024, 4:06 PM
Unknown Object (File)
Oct 22 2024, 7:58 AM
Unknown Object (File)
Oct 19 2024, 5:01 PM
View All 48 Files
Subscribers
bdrewery
emaste

Details

Reviewers
emaste
Commits
rGe0281204cce3: Add clarifying comments
Summary

Expand on the terse comments for where each of these files is used.

Also, remove some mips specific stuff from bsd.opts.mk while I'm there.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp created this revision.Jan 1 2022, 11:51 PM
Herald added subscribers: emaste, bdrewery. · View Herald TranscriptJan 1 2022, 11:51 PM
imp requested review of this revision.Jan 1 2022, 11:51 PM
Harbormaster completed remote builds in B43669: Diff 100813.Jan 1 2022, 11:51 PM
imp added a reviewer: emaste.Jan 1 2022, 11:51 PM
emaste accepted this revision.Jan 2 2022, 12:41 AM

Thanks this does clarify the purpose of the files. I'd think the MIPS changes should be an independent commit tho.

share/mk/bsd.opts.mk
102

This should probably go in by itself?

This revision is now accepted and ready to land.Jan 2 2022, 12:41 AM
imp added inline comments.Jan 2 2022, 12:57 AM
share/mk/bsd.opts.mk
102

Agreed. I'd come to the same conclusion...

Closed by commit rGe0281204cce3: Add clarifying comments (authored by imp). · Explain WhyJan 2 2022, 7:32 AM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rGe0281204cce3: Add clarifying comments.

Revision Contents
Changeset List

PathSize
share/
mk/
5 lines
4 lines
sys/
conf/
5 lines

Diff 100820

View Options

share/mk/bsd.opts.mk

Loading...
View Options

share/mk/src.opts.mk

Loading...
View Options

sys/conf/kern.opts.mk

Loading...