Page MenuHomeFreeBSD

Remove more quotes around Makefile .error/.warn/.info strings.
ClosedPublic

Authored by jhb on Feb 4 2022, 8:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 26, 10:58 AM
Unknown Object (File)
Tue, Jan 14, 9:59 PM
Unknown Object (File)
Tue, Jan 14, 7:58 AM
Unknown Object (File)
Dec 14 2024, 9:22 AM
Unknown Object (File)
Dec 13 2024, 9:39 AM
Unknown Object (File)
Nov 30 2024, 11:26 PM
Unknown Object (File)
Nov 13 2024, 3:22 PM
Unknown Object (File)
Nov 13 2024, 1:40 PM
Subscribers

Diff Detail

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

Event Timeline

jhb requested review of this revision.Feb 4 2022, 8:56 PM

I thought we'd already done this.

share/mk/netbsd-tests.test.mk
9

Does this need $$? It was there before, but do you want to suggest it in terms of SRCTOP (in which case the syntax is wrong for some shells) or with the current value of srctop?

This revision is now accepted and ready to land.Feb 4 2022, 9:01 PM

Apparently I missed some, I might have just grepped Makefiles

share/mk/bsd.dep.mk
205–206

We can remove the space before \

share/mk/netbsd-tests.test.mk
9

My guess is that we want it to be emitted as a literal ${SRCTOP} not expanded, in that the person who encounters this error should set TESTSRC=${SRCTOP}/... in their Makefile.

jhb marked an inline comment as done.Feb 5 2022, 12:01 AM
jhb added inline comments.
share/mk/netbsd-tests.test.mk
9

Yeah, I think what Ed said.