* [PATCH] meson: fix "static build" entry in summary
@ 2023-06-07 9:03 Paolo Bonzini
2023-06-07 9:08 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 4+ messages in thread
From: Paolo Bonzini @ 2023-06-07 9:03 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial
Fixes: a0cbd2e8496 ("meson: use prefer_static option", 2023-05-18)
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 553c8e0b9c5..34d0444ffbb 100644
--- a/meson.build
+++ b/meson.build
@@ -4088,7 +4088,7 @@ summary_info += {'QEMU_LDFLAGS': ' '.join(qemu_ldflags)}
summary_info += {'profiler': get_option('profiler')}
summary_info += {'link-time optimization (LTO)': get_option('b_lto')}
summary_info += {'PIE': get_option('b_pie')}
-summary_info += {'static build': config_host.has_key('CONFIG_STATIC')}
+summary_info += {'static build': get_optin('prefer_static')}
summary_info += {'malloc trim support': has_malloc_trim}
summary_info += {'membarrier': have_membarrier}
summary_info += {'debug graph lock': get_option('debug_graph_lock')}
--
2.40.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: fix "static build" entry in summary
2023-06-07 9:03 [PATCH] meson: fix "static build" entry in summary Paolo Bonzini
@ 2023-06-07 9:08 ` Philippe Mathieu-Daudé
2023-06-07 9:26 ` Paolo Bonzini
0 siblings, 1 reply; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-06-07 9:08 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-trivial
On 7/6/23 11:03, Paolo Bonzini wrote:
> Fixes: a0cbd2e8496 ("meson: use prefer_static option", 2023-05-18)
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> meson.build | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meson.build b/meson.build
> index 553c8e0b9c5..34d0444ffbb 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -4088,7 +4088,7 @@ summary_info += {'QEMU_LDFLAGS': ' '.join(qemu_ldflags)}
> summary_info += {'profiler': get_option('profiler')}
> summary_info += {'link-time optimization (LTO)': get_option('b_lto')}
> summary_info += {'PIE': get_option('b_pie')}
> -summary_info += {'static build': config_host.has_key('CONFIG_STATIC')}
> +summary_info += {'static build': get_optin('prefer_static')}
Typo 'get_option'.
> summary_info += {'malloc trim support': has_malloc_trim}
> summary_info += {'membarrier': have_membarrier}
> summary_info += {'debug graph lock': get_option('debug_graph_lock')}
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: fix "static build" entry in summary
2023-06-07 9:08 ` Philippe Mathieu-Daudé
@ 2023-06-07 9:26 ` Paolo Bonzini
2023-06-07 9:30 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 4+ messages in thread
From: Paolo Bonzini @ 2023-06-07 9:26 UTC (permalink / raw)
To: Philippe Mathieu-Daudé; +Cc: qemu-devel, qemu-trivial
On Wed, Jun 7, 2023 at 11:09 AM Philippe Mathieu-Daudé
<philmd@linaro.org> wrote:
>
> On 7/6/23 11:03, Paolo Bonzini wrote:
> > Fixes: a0cbd2e8496 ("meson: use prefer_static option", 2023-05-18)
> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> > ---
> > meson.build | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meson.build b/meson.build
> > index 553c8e0b9c5..34d0444ffbb 100644
> > --- a/meson.build
> > +++ b/meson.build
> > @@ -4088,7 +4088,7 @@ summary_info += {'QEMU_LDFLAGS': ' '.join(qemu_ldflags)}
> > summary_info += {'profiler': get_option('profiler')}
> > summary_info += {'link-time optimization (LTO)': get_option('b_lto')}
> > summary_info += {'PIE': get_option('b_pie')}
> > -summary_info += {'static build': config_host.has_key('CONFIG_STATIC')}
> > +summary_info += {'static build': get_optin('prefer_static')}
>
> Typo 'get_option'.
Yeah, sent after testing but before committing the obvious fix. :(
Paolo
> > summary_info += {'malloc trim support': has_malloc_trim}
> > summary_info += {'membarrier': have_membarrier}
> > summary_info += {'debug graph lock': get_option('debug_graph_lock')}
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: fix "static build" entry in summary
2023-06-07 9:26 ` Paolo Bonzini
@ 2023-06-07 9:30 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-06-07 9:30 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: qemu-devel, qemu-trivial
On 7/6/23 11:26, Paolo Bonzini wrote:
> On Wed, Jun 7, 2023 at 11:09 AM Philippe Mathieu-Daudé
> <philmd@linaro.org> wrote:
>>
>> On 7/6/23 11:03, Paolo Bonzini wrote:
>>> Fixes: a0cbd2e8496 ("meson: use prefer_static option", 2023-05-18)
>>> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>>> ---
>>> meson.build | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/meson.build b/meson.build
>>> index 553c8e0b9c5..34d0444ffbb 100644
>>> --- a/meson.build
>>> +++ b/meson.build
>>> @@ -4088,7 +4088,7 @@ summary_info += {'QEMU_LDFLAGS': ' '.join(qemu_ldflags)}
>>> summary_info += {'profiler': get_option('profiler')}
>>> summary_info += {'link-time optimization (LTO)': get_option('b_lto')}
>>> summary_info += {'PIE': get_option('b_pie')}
>>> -summary_info += {'static build': config_host.has_key('CONFIG_STATIC')}
>>> +summary_info += {'static build': get_optin('prefer_static')}
>>
>> Typo 'get_option'.
>
> Yeah, sent after testing but before committing the obvious fix. :(
So with the typo fixed:
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
:)
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-06-07 9:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-07 9:03 [PATCH] meson: fix "static build" entry in summary Paolo Bonzini
2023-06-07 9:08 ` Philippe Mathieu-Daudé
2023-06-07 9:26 ` Paolo Bonzini
2023-06-07 9:30 ` Philippe Mathieu-Daudé
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).