mbox series

[0/2] tools/bootconfig: allow overriding CFLAGS and LDFLAGS

Message ID 20250328183858.1417835-1-bmasney@redhat.com (mailing list archive)
Headers show
Series tools/bootconfig: allow overriding CFLAGS and LDFLAGS | expand

Message

Brian Masney March 28, 2025, 6:38 p.m. UTC 
Allow the user to override the CFLAGS and LDFLAGS similar to what's
currently done on other kernel tools. These changes allows bootconfig to
be compiled with the hardened compiler and linker flags.

Brian Masney (2):
  tools/bootconfig: allow overriding CFLAGS assignment
  tools/bootconfig: specify LDFLAGS as an argument to CC

 tools/bootconfig/Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Masami Hiramatsu (Google) March 31, 2025, 12:23 p.m. UTC | #1 
On Fri, 28 Mar 2025 14:38:56 -0400
Brian Masney <bmasney@redhat.com> wrote:

> Allow the user to override the CFLAGS and LDFLAGS similar to what's
> currently done on other kernel tools. These changes allows bootconfig to
> be compiled with the hardened compiler and linker flags.
> 
> Brian Masney (2):
>   tools/bootconfig: allow overriding CFLAGS assignment
>   tools/bootconfig: specify LDFLAGS as an argument to CC
> 

Both looks good to me. Let me pick those up.

Thanks!

>  tools/bootconfig/Makefile | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> -- 
> 2.47.0
>
Brian Masney April 10, 2025, 3:36 p.m. UTC | #2 
Hi Masami,

On Mon, Mar 31, 2025 at 8:23 AM Masami Hiramatsu <mhiramat@kernel.org> wrote:
> On Fri, 28 Mar 2025 14:38:56 -0400
> Brian Masney <bmasney@redhat.com> wrote:
>
> > Allow the user to override the CFLAGS and LDFLAGS similar to what's
> > currently done on other kernel tools. These changes allows bootconfig to
> > be compiled with the hardened compiler and linker flags.
> >
> > Brian Masney (2):
> >   tools/bootconfig: allow overriding CFLAGS assignment
> >   tools/bootconfig: specify LDFLAGS as an argument to CC
> >
>
> Both looks good to me. Let me pick those up.

Should these show up in linux-next by now? I see them in your tree at:

https://web.git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace.git/log/?h=bootconfig/tools

But not in the for-next branch.

Thanks,
Brian