test-includes: use the same header set for badfiles.inc as for testing
ClosedPublic
Actions

Authored by kp on Dec 16 2021, 9:33 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, May 6, 12:33 AM

	Unknown Object (File)
	Wed, May 1, 10:25 PM

	Unknown Object (File)
	Apr 11 2024, 1:53 PM

	Unknown Object (File)
	Mar 14 2024, 9:29 AM

	Unknown Object (File)
	Mar 14 2024, 9:28 AM

	Unknown Object (File)
	Mar 14 2024, 9:24 AM

	Unknown Object (File)
	Mar 14 2024, 9:16 AM

	Unknown Object (File)
	Mar 4 2024, 8:08 AM

View All 40 Files

Subscribers

None

Details

Reviewers

imp

Group Reviewers

network
pfsense

Commits

rG2f3c018ba8e8: test-includes: use the same header set for badfiles.inc as for testing

Summary

This ensures we don't end up listing files we've excluded (e.g. those
starting with _).
Add a slight hack to preserve the existing order (sys, then net) in
badfiles.inc.

Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp requested review of this revision.Dec 16 2021, 9:33 AM

kp created this revision.

kp added a child revision: D33506: test-includes: update badfiles.inc.

Harbormaster completed remote builds in B43412: Diff 100133.Dec 16 2021, 9:34 AM

kp added a parent revision: D33504: pf: make if_pfsync.h self-contained.Dec 16 2021, 9:34 AM

Nice catch

This revision is now accepted and ready to land.Dec 16 2021, 9:36 AM

Closed by commit rG2f3c018ba8e8: test-includes: use the same header set for badfiles.inc as for testing (authored by kp). · Explain WhyDec 17 2021, 11:39 AM

This revision was automatically updated to reflect the committed changes.

kp added a commit: rG2f3c018ba8e8: test-includes: use the same header set for badfiles.inc as for testing.

Revision Contents
Changeset List

Path

Size

tools/

build/

test-includes/

Makefile

4 lines

Diff 100210

View Options

tools/build/test-includes/Makefile

test-includes: use the same header set for badfiles.inc as for testingClosedPublicActions