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)
Mar 7 2024, 12:19 PM
Unknown Object (File)
Feb 24 2024, 9:03 AM
Unknown Object (File)
Jan 16 2024, 11:16 AM
Unknown Object (File)
Dec 23 2023, 3:08 AM
Unknown Object (File)
Dec 19 2023, 7:42 PM
Unknown Object (File)
Dec 12 2023, 3:44 AM
Unknown Object (File)
Sep 21 2023, 10:17 PM
Unknown Object (File)
Sep 16 2023, 12:20 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.