diff mbox series

[v1,2/6] meson.build: fix cosmetics of compiler display

Message ID 20210527160319.19834-3-alex.bennee@linaro.org
State Superseded
Headers show
Series testing/next queue (signal, meson, acceptance tags) | expand

Commit Message

Alex Bennée May 27, 2021, 4:03 p.m. UTC 
If you specify something like --cc="ccache gcc" on your configure line
the summary output misses the rest of the cmd_array. Do some string
joining to make it complete.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Tested-by: Thomas Huth <thuth@redhat.com>

---
 meson.build | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

-- 
2.20.1

Comments

Richard Henderson May 27, 2021, 5:38 p.m. UTC | #1 
On 5/27/21 9:03 AM, Alex Bennée wrote:
> If you specify something like --cc="ccache gcc" on your configure line

> the summary output misses the rest of the cmd_array. Do some string

> joining to make it complete.

> 

> Signed-off-by: Alex Bennée<alex.bennee@linaro.org>

> Tested-by: Thomas Huth<thuth@redhat.com>

> ---

>   meson.build | 8 ++++----

>   1 file changed, 4 insertions(+), 4 deletions(-)


Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~
Philippe Mathieu-Daudé May 28, 2021, 11:05 a.m. UTC | #2 
On 5/27/21 6:03 PM, Alex Bennée wrote:
> If you specify something like --cc="ccache gcc" on your configure line

> the summary output misses the rest of the cmd_array. Do some string

> joining to make it complete.

> 

> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

> Tested-by: Thomas Huth <thuth@redhat.com>

> ---

>  meson.build | 8 ++++----

>  1 file changed, 4 insertions(+), 4 deletions(-)


Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
diff mbox series

Patch

diff --git a/meson.build b/meson.build
index 20d7035e44..3f065f53f2 100644
--- a/meson.build
+++ b/meson.build
@@ -2515,15 +2515,15 @@  summary(summary_info, bool_yn: true, section: 'Configurable features')
 summary_info = {}
 summary_info += {'host CPU':          cpu}
 summary_info += {'host endianness':   build_machine.endian()}
-summary_info += {'C compiler':        meson.get_compiler('c').cmd_array()[0]}
-summary_info += {'Host C compiler':   meson.get_compiler('c', native: true).cmd_array()[0]}
+summary_info += {'C compiler':        ' '.join(meson.get_compiler('c').cmd_array())}
+summary_info += {'Host C compiler':   ' '.join(meson.get_compiler('c', native: true).cmd_array())}
 if link_language == 'cpp'
-  summary_info += {'C++ compiler':      meson.get_compiler('cpp').cmd_array()[0]}
+  summary_info += {'C++ compiler':    ' '.join(meson.get_compiler('cpp').cmd_array())}
 else
   summary_info += {'C++ compiler':      false}
 endif
 if targetos == 'darwin'
-  summary_info += {'Objective-C compiler': meson.get_compiler('objc').cmd_array()[0]}
+  summary_info += {'Objective-C compiler': ' '.join(meson.get_compiler('objc').cmd_array())}
 endif
 if targetos == 'windows'
   if 'WIN_SDK' in config_host