Paths

Table of Contentst

Differential D50234

fts: Add tests for most FTS options.
ClosedPublic
Actions

Authored by des on May 7 2025, 11:32 AM.

Tags

None

Referenced Files

	F163102648: D50234.id155044.diff
	Mon, Jul 20, 2:44 AM

	Unknown Object (File)
	Sat, Jul 18, 12:40 PM

	Unknown Object (File)
	Sat, Jul 18, 9:36 AM

	Unknown Object (File)
	Sat, Jul 18, 5:09 AM

	Unknown Object (File)
	Sat, Jul 18, 4:10 AM

	Unknown Object (File)
	Fri, Jul 17, 9:45 AM

	Unknown Object (File)
	Fri, Jul 10, 11:14 PM

	Unknown Object (File)
	Thu, Jul 9, 5:01 PM

View All 96 Files

Subscribers

Details

Reviewers

imp
kevans

Group Reviewers

Commits

rGd2da6ed98494: fts: Add tests for most FTS options.

Summary

Sponsored by: Klara, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 63988
Build 60872: arc lint + arc unit

Event Timeline

des created this revision.May 7 2025, 11:32 AM

Herald added a subscriber: imp. · View Herald TranscriptMay 7 2025, 11:32 AM

des requested review of this revision.May 7 2025, 11:32 AM

Harbormaster completed remote builds in B63988: Diff 155044.May 7 2025, 11:32 AM

des added a parent revision: D50233: fts: Add FTS_COMFOLLOWDIR and FTS_NOSTAT_TYPE..May 7 2025, 11:32 AM

des added a child revision: D50235: fts: Give the blocks test a description..

imp accepted this revision.May 7 2025, 8:40 PM

This revision is now accepted and ready to land.May 7 2025, 8:40 PM

kevans accepted this revision.May 8 2025, 1:25 AM

kevans added a subscriber: kevans.

kevans added inline comments.

lib/libc/tests/gen/fts_options.c
52	That's cute, I kind of wish we had some atf-c(3) support for this instead to abstract away that particular detail. That's fine for now, unless you didn't intend to leave it in.

des marked an inline comment as done.May 8 2025, 6:23 AM

des added inline comments.

lib/libc/tests/gen/fts_options.c
52	On the contrary, this is the compromise I arrived at to avoid removing the `fprintf()` at the top of the loop in `fts_options_test()`, because I don't want it in regular test output, but this test is hopeless to debug without it.

des marked an inline comment as done.May 8 2025, 6:23 AM

Closed by commit rGd2da6ed98494: fts: Add tests for most FTS options. (authored by des). · Explain WhyMay 8 2025, 2:30 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGd2da6ed98494: fts: Add tests for most FTS options..

Revision Contents
Changeset List

Path

Size

lib/

libc/

tests/

gen/

1 line

454 lines

Diff 155044

lib/libc/tests/gen/Makefile

Loading...

lib/libc/tests/gen/fts_options.c

Loading...