diff mbox series

[v2] buildsys: Only display Objective-C information when Objective-C is used

Message ID 20231009093812.52915-1-philmd@linaro.org
State New
Headers show
Series [v2] buildsys: Only display Objective-C information when Objective-C is used | expand

Commit Message

Philippe Mathieu-Daudé Oct. 9, 2023, 9:38 a.m. UTC
When configuring with '--disable-cocoa --disable-coreaudio'
on Darwin, we get:

 meson.build:4081:58: ERROR: Tried to access compiler for language "objc", not specified for host machine.
 meson.build:4097:47: ERROR: Tried to access unknown option 'objc_args'.

Instead of unconditionally display Objective-C informations
on Darwin, display them when Objective-C is discovered.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
v2: Emit 'false' (Akihiko)
---
 meson.build | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Akihiko Odaki Oct. 9, 2023, 9:39 a.m. UTC | #1
On 2023/10/09 18:38, Philippe Mathieu-Daudé wrote:
> When configuring with '--disable-cocoa --disable-coreaudio'
> on Darwin, we get:
> 
>   meson.build:4081:58: ERROR: Tried to access compiler for language "objc", not specified for host machine.
>   meson.build:4097:47: ERROR: Tried to access unknown option 'objc_args'.
> 
> Instead of unconditionally display Objective-C informations
> on Darwin, display them when Objective-C is discovered.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> v2: Emit 'false' (Akihiko)
> ---
>   meson.build | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index 3bb64b536c..567c1c9add 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -4074,8 +4074,10 @@ if 'cpp' in all_languages
>   else
>     summary_info += {'C++ compiler':      false}
>   endif
> -if targetos == 'darwin'
> +if 'objc' in all_languages
>     summary_info += {'Objective-C compiler': ' '.join(meson.get_compiler('objc').cmd_array())}
> +else
> +  summary_info += {'Objective-C compiler': false}
>   endif
>   option_cflags = (get_option('debug') ? ['-g'] : [])
>   if get_option('optimization') != 'plain'
> @@ -4085,7 +4087,7 @@ summary_info += {'CFLAGS':            ' '.join(get_option('c_args') + option_cfl
>   if 'cpp' in all_languages
>     summary_info += {'CXXFLAGS':        ' '.join(get_option('cpp_args') + option_cflags)}
>   endif
> -if targetos == 'darwin'
> +if 'objc' in all_languages
>     summary_info += {'OBJCFLAGS':       ' '.join(get_option('objc_args') + option_cflags)}
>   endif
>   link_args = get_option('c_link_args')

Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
diff mbox series

Patch

diff --git a/meson.build b/meson.build
index 3bb64b536c..567c1c9add 100644
--- a/meson.build
+++ b/meson.build
@@ -4074,8 +4074,10 @@  if 'cpp' in all_languages
 else
   summary_info += {'C++ compiler':      false}
 endif
-if targetos == 'darwin'
+if 'objc' in all_languages
   summary_info += {'Objective-C compiler': ' '.join(meson.get_compiler('objc').cmd_array())}
+else
+  summary_info += {'Objective-C compiler': false}
 endif
 option_cflags = (get_option('debug') ? ['-g'] : [])
 if get_option('optimization') != 'plain'
@@ -4085,7 +4087,7 @@  summary_info += {'CFLAGS':            ' '.join(get_option('c_args') + option_cfl
 if 'cpp' in all_languages
   summary_info += {'CXXFLAGS':        ' '.join(get_option('cpp_args') + option_cflags)}
 endif
-if targetos == 'darwin'
+if 'objc' in all_languages
   summary_info += {'OBJCFLAGS':       ' '.join(get_option('objc_args') + option_cflags)}
 endif
 link_args = get_option('c_link_args')