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)
Mon, Nov 24, 5:06 PM
Unknown Object (File)
Sat, Nov 15, 4:36 PM
Unknown Object (File)
Sat, Nov 15, 4:36 PM
Unknown Object (File)
Sat, Nov 15, 2:29 PM
Unknown Object (File)
Sat, Nov 15, 12:22 PM
Unknown Object (File)
Wed, Nov 5, 9:31 PM
Unknown Object (File)
Wed, Nov 5, 5:29 PM
Unknown Object (File)
Wed, Nov 5, 4:25 PM
View All 85 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...