[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: Do not display host compilers as cro
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH] configure: Do not display host compilers as cross-compilers |
Date: |
Wed, 4 Jul 2018 16:48:31 +0100 |
User-agent: |
Mutt/1.10.0 (2018-05-17) |
On Wed, Jul 04, 2018 at 04:41:30PM +0100, Alex Bennée wrote:
>
> Philippe Mathieu-Daudé <address@hidden> writes:
>
> > This silent (since d75402b5ee2):
> >
> > ./configure
> > [...]
> > NOTE: cross-compilers enabled: 'cc'
> >
> > Reported-by: Dr. David Alan Gilbert <address@hidden>
> > Reported-by: Daniel P. Berrangé <address@hidden>
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > ---
> > configure | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/configure b/configure
> > index b99ebdde24..fbb53ab6bb 100755
> > --- a/configure
> > +++ b/configure
> > @@ -7328,7 +7328,7 @@ echo "QEMU_CFLAGS+=$cflags" >> $config_target_mak
> >
> > done # for target in $targets
> >
> > -if test -n "$enabled_cross_compilers"; then
> > +if test -n "$target_compiler" -a -n "$enabled_cross_compilers"; then
> > echo
> > echo "NOTE: cross-compilers enabled: $enabled_cross_compilers"
> > fi
>
> I had also added:
>
> - echo "NOTE: cross-compilers enabled: $enabled_cross_compilers"
> + echo "NOTE: compilers enabled for TCG tests: $enabled_cross_compilers"[
>
> to make things clearer.
I think we should also 's/NOTE: //', because everything printed by
configure at the end is a note. Also get rid of the extra blank
line before it.
>
> Acked-by: Alex Bennée <address@hidden>
>
>
> --
> Alex Bennée
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
Re: [Qemu-devel] [PATCH] configure: Do not display host compilers as cross-compilers, no-reply, 2018/07/04