Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recent dev vs. tools/build/test-includes #1321

Closed
nwf opened this issue Mar 17, 2022 · 4 comments
Closed

Recent dev vs. tools/build/test-includes #1321

nwf opened this issue Mar 17, 2022 · 4 comments

Comments

@nwf
Copy link
Member

nwf commented Mar 17, 2022

FWIW, a clean build of dev fails to make much progress on my machine because all the sys/_*.h files now get included and all, quite deliberately, cause the compiler to bomb out with "don't do that" errors.

I believe this an upstream FreeBSD issue and is, ultimately, because

HDRS!= (cd ${SRCTOP}/sys; ls sys/[^_]*.h | sort ; ls net*/[^_]*.h | sort)

is not actually spec conformant. In particular, X/Open CAE Specification Commands and Utilities Issue 4, Version 2 sayeth:

The description of basic regular expression bracket expressions in the XBD specification, Section
7.3.5, RE Bracket Expression also applies to the pattern bracket expression, except that the
exclamation-mark character (!) replaces the circumflex character (ˆ) in its role in a non-matching
list in the regular expression notation. A bracket expression starting with an unquoted
circumflex character produces unspecified results.

There must be something about my setup that elicits these "unspecified results", though I hesitate to speculate which of its many, many "charming features", exactly, that might be.

@brooksdavis
Copy link
Member

brooksdavis commented Mar 17, 2022

From the FreeBSD manpage:

The character class may be complemented by making an exclamation
point ('!') the first character of the character class. A caret ('^')
has the same effect but is non-standard.

Could you test with ! in place of ^? If that works I can prod Warner.

@nwf
Copy link
Member Author

nwf commented Mar 17, 2022

That (! and not ^) seems to work fine.

@brooksdavis
Copy link
Member

Submitted for review https://reviews.freebsd.org/D34597

freebsd-git pushed a commit to freebsd/freebsd-src that referenced this issue Mar 22, 2022
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:		CTSRD-CHERI/cheribsd#1321
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D34597
brooksdavis added a commit that referenced this issue Apr 26, 2022
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:		#1321
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D34597
brooksdavis added a commit that referenced this issue Apr 26, 2022
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:		#1321
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D34597

(cherry picked from commit 44b36b8)
bsdjhb pushed a commit that referenced this issue Apr 26, 2022
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:		#1321
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D34597

(cherry picked from commit 44b36b8)
@brooksdavis
Copy link
Member

Merged to dev

bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this issue Jun 6, 2022
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:		CTSRD-CHERI#1321
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D34597
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants