Add a guard for broken SUBDIR.${MK_FOO} use
ClosedPublic
Actions

Authored by brooks on Nov 13 2020, 10:36 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jul 21, 7:13 AM

	Unknown Object (File)
	Tue, Jul 21, 6:34 AM

	Unknown Object (File)
	Mon, Jul 13, 6:39 PM

	Unknown Object (File)
	Sat, Jul 11, 8:55 PM

	Unknown Object (File)
	Tue, Jun 23, 11:06 PM

	Unknown Object (File)
	Tue, Jun 23, 3:03 AM

	Unknown Object (File)
	May 17 2026, 7:44 PM

	Unknown Object (File)
	Apr 29 2026, 10:16 AM

View All Files

Subscribers

emaste

imp

Details

Reviewers

bdrewery
arichardson

Commits

rS367735: Add a guard for broken SUBDIR.${MK_FOO} use

Summary

Check for the variable SUBDIR. and error as it usually means someone
forgot to include src.opts.mk.

This guard from CheriBSD found the bug in D27210.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 34791
Build 31840: arc lint + arc unit

Event Timeline

brooks created this revision.Nov 13 2020, 10:36 PM

Herald added a subscriber: emaste. · View Herald TranscriptNov 13 2020, 10:36 PM

brooks requested review of this revision.Nov 13 2020, 10:36 PM

Harbormaster completed remote builds in B34791: Diff 79528.Nov 13 2020, 10:36 PM

Brilliant!

This revision is now accepted and ready to land.Nov 13 2020, 10:56 PM

git blame points at https://github.com/CTSRD-CHERI/cheribsd/commit/4d2d3d6dc7e7e38ce6403654f0f998e79ca4e0c4

Looks like I forgot to create the upstream review.

Closed by commit rS367735: Add a guard for broken SUBDIR.${MK_FOO} use (authored by brooks). · Explain WhyNov 16 2020, 7:15 PM

This revision was automatically updated to reflect the committed changes.

brooks added a commit: rS367735: Add a guard for broken SUBDIR.${MK_FOO} use.

Herald added a subscriber: imp. · View Herald TranscriptNov 16 2020, 7:15 PM

Revision Contents
Changeset List

Path

Size

share/

mk/

bsd.subdir.mk

6 lines

Diff 79528

View Options

Add a guard for broken SUBDIR.${MK_FOO} useClosedPublicActions