Paths

Table of Contentst

Differential D50480

share/mk: improve CFLAGS description
Needs ReviewPublic
Actions

Authored by brooks on May 22 2025, 10:34 PM.

Tags

None

Referenced Files

	F133219717: D50480.id155950.diff
	Fri, Oct 24, 2:08 AM

	Unknown Object (File)
	Tue, Oct 7, 5:36 PM

	Unknown Object (File)
	Fri, Oct 3, 4:41 PM

	Unknown Object (File)
	Fri, Sep 26, 12:51 PM

	Unknown Object (File)
	Thu, Sep 25, 7:22 PM

	Unknown Object (File)
	Thu, Sep 25, 1:09 AM

	Unknown Object (File)
	Sep 24 2025, 5:12 AM

	Unknown Object (File)
	Sep 18 2025, 7:18 AM

View All 55 Files

Subscribers

Details

Reviewers

emaste
jhb

Summary

Also document CXXFLAGS.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 64371
Build 61255: arc lint + arc unit

Event Timeline

brooks created this revision.May 22 2025, 10:34 PM

Herald added subscribers: imp, bdrewery. · View Herald TranscriptMay 22 2025, 10:34 PM

brooks requested review of this revision.May 22 2025, 10:34 PM

Harbormaster completed remote builds in B64371: Diff 155899.May 22 2025, 10:34 PM

brooks added a parent revision: D50479: share/mk: make *ADD and *FLAGS descriptions common.May 22 2025, 10:34 PM

brooks added a child revision: D50481: share/mk/bsd.README: refer to the linker, not the loader.

emaste added inline comments.May 22 2025, 11:26 PM

share/mk/bsd.README
136	Should we (in an ideal world, absent legacy and backwards compatibility considerations) really have `CPPFLAGS` and reserve `CFLAGS` specifically for C?

brooks added inline comments.May 22 2025, 11:40 PM

share/mk/bsd.README
136	Certainly something C specific would be good, but we also need something to pass all the common flags. We've got a lot of ABI stuff in CFLAGS via bsd.cpu.mk. Likewise we add -target and --sysroot to XCFLAGS so they end up in CFLAGS.

jhb added inline comments.May 23 2025, 5:34 PM

share/mk/bsd.README
136	Maybe either "things that are run though the C preprocessor" or "Flags to the compiler when compiling most things processed by the C preprocessor."?
151–152	This seems more consistent with the wording used for AFLAGS and CFLAGS?

Rephrased with some help from @jhb

Harbormaster completed remote builds in B64396: Diff 155950.May 23 2025, 10:59 PM

Spell version correctly

Harbormaster completed remote builds in B64397: Diff 155951.May 23 2025, 11:01 PM

brooks marked 3 inline comments as done.May 23 2025, 11:11 PM

Revision Contents
Changeset List

Path

Size

share/

mk/

7 lines

Diff 155899

share/mk/bsd.README

Loading...