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, Sep 30, 3:17 PM
Unknown Object (File)
Sun, Sep 28, 3:30 PM
Unknown Object (File)
Sun, Sep 28, 1:43 PM
Unknown Object (File)
Fri, Sep 26, 5:50 PM
Unknown Object (File)
Aug 22 2025, 8:37 AM
Unknown Object (File)
Aug 22 2025, 8:36 AM
Unknown Object (File)
Aug 22 2025, 8:36 AM
Unknown Object (File)
Aug 22 2025, 8:24 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?