Re: [Buildroot] [PATCH 1/1] package/ntpsec: fix build on xtensa

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Dario,

On Sat, 27 Jul 2024 11:40:29 +0200
Dario Binacchi <dario.binacchi@amarulasolutions.com> wrote:

> Fix the following build failure on xtensa:
> 
> Waf: Entering directory `/home/buildroot/instance-0/output-1/build/ntpsec-1.2.3/build/host'
> [1/2] Processing ntpd/ntp_parser.y
> [2/2] Compiling build/host/ntpd/ntp_parser.tab.c
> gcc: error: unrecognized command-line option '-mlongcalls'
> gcc: error: unrecognized command-line option '-mauto-litpools'

Thanks for the patch, but I'm pretty sure this is not the right fix.
You can see from the error message above that what is happening here is
that those flags, which are target CFLAGS, are passed to the host
compiler "gcc".

So your patch is not correct in several ways:

 - It will not solve the problem of other target CFLAGS used for other
   CPU architectures, which are also not supported by the host gcc
   compiler

 - It drops those target CFLAGS from the build of target code, which is
   not correct, as if some flags are present, there's a good reason for
   that.

So, the correct fix is to look in more details into the ntpsec build
system, and make sure it doesn't use target CFLAGS when building host
code.

Could you have a look into this?

Thanks a lot,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot