Page MenuHomeFreeBSD

Makefile.asm: Simplify variable expansions in generated headers
ClosedPublic

Authored by jhb on Aug 24 2023, 7:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 5:58 PM
Unknown Object (File)
Nov 23 2024, 3:15 PM
Unknown Object (File)
Nov 23 2024, 2:57 PM
Unknown Object (File)
Nov 23 2024, 2:29 PM
Unknown Object (File)
Nov 23 2024, 2:09 PM
Unknown Object (File)
Nov 23 2024, 11:53 AM
Unknown Object (File)
Nov 22 2024, 10:38 PM
Unknown Object (File)
Nov 13 2024, 5:32 AM
Subscribers

Diff Detail

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

Event Timeline

I'd consider defining a variable instead of repeating ${.ALLSRC:T}, or even keep the full message in a variable, so that future changes are easier.

so just trim them to using :T alone.

Extra "to"?

This revision is now accepted and ready to land.Aug 28 2023, 4:26 PM
emaste added a subscriber: jkim.

I'd consider defining a variable instead of repeating ${.ALLSRC:T}, or even keep the full message in a variable, so that future changes are easier.

Well, some cases use ${.IMPSRC:T} and others use ${.ALLSRC:T}. It may make sense to define a variable for the rest of the message, in particular to simplify Ed's request to use @generated in the comment.

so just trim them to using :T alone.

Extra "to"?

Hmm, I don't think so, but it is a bit wordy. "... again, just use :T alone." might be clearer.

To me this looks tidier and also more sensible than before; indeed, why did it set the suffix to .pl only to remove it and add it again?