[PATCH] kbuild: Strip runtime const RELA sections correctly

Linus Torvalds torvalds at linux-foundation.org
Fri Jan 10 10:32:54 PST 2025


On Fri, 10 Jan 2025 at 10:11, Ard Biesheuvel <ardb at kernel.org> wrote:
>
> I suppose this is a consequence of 8962b6b475bddc ("kbuild: print
> short log in addition to the whole command with V=1") which introduced
> an $(if ) where the else branch is simply ':' and so it always
> succeeds.

Hmm. Odd. I don't see why that part of the commit exists, and you're
right, that seems like a bad idea.

And removing that odd $(if..) and making it just do

   cmd = @set -e; $($(quiet)log_print) $(delete-on-interrupt) $(cmd_$(1))

doesn't seem to have any obvious negative effects.

I'm probably missing some obvious reason why that $(if..) was added,
it's been that way for two years now.

           Linus



More information about the linux-riscv mailing list