Page MenuHomeFreeBSD
Differential D34597

test-includes: improve portability
ClosedPublic
Actions

Authored by brooks on Mar 17 2022, 11:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Aug 9, 4:22 PM
Unknown Object (File)
Sat, Aug 8, 7:51 PM
Unknown Object (File)
Fri, Aug 7, 8:57 PM
Unknown Object (File)
Fri, Aug 7, 6:11 PM
Unknown Object (File)
Wed, Jul 29, 12:35 AM
Unknown Object (File)
Wed, Jul 22, 9:27 PM
Unknown Object (File)
Tue, Jul 21, 4:28 PM
Unknown Object (File)
Tue, Jul 21, 12:13 AM
View All Files
Subscribers
None

Details

Reviewers
imp
Commits
rG44b36b80f578: test-includes: improve portability
Summary

FreeBSD's shell permits [^bar] to match characters not in [bar] like
common regular expressions. This is non-portable and '!' should be
used in place of '^' for portability.

Reported by: Nathaniel Wesley Filardo
Issue: https://github.com/CTSRD-CHERI/cheribsd/issues/1321

Diff Detail

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

Event Timeline

brooks requested review of this revision.Mar 17 2022, 11:47 PM
brooks created this revision.
Harbormaster completed remote builds in B44818: Diff 103971.Mar 17 2022, 11:47 PM
imp added inline comments.Mar 18 2022, 3:52 AM
tools/build/test-includes/Makefile
19

Looks good, or it could be [a-zA-z]*.h too...

imp accepted this revision.Mar 18 2022, 3:53 AM
This revision is now accepted and ready to land.Mar 18 2022, 3:53 AM
Closed by commit rG44b36b80f578: test-includes: improve portability (authored by brooks). · Explain WhyMar 22 2022, 8:24 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rG44b36b80f578: test-includes: improve portability.

Revision Contents
Changeset List

PathSize
tools/
build/
test-includes/
2 lines

Diff 104097

View Options

tools/build/test-includes/Makefile

Loading...