All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Tokarev <mjt@tls.msk.ru>
To: Brad Smith <brad@comstyle.com>
Cc: qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [Qemu] [PATCH for 1.5] ui/gtk.c: Fix *BSD build of Gtk+ UI
Date: Wed, 05 Jun 2013 14:13:26 +0400	[thread overview]
Message-ID: <51AF0F46.3020002@msgid.tls.msk.ru> (raw)
In-Reply-To: <20130522223817.GB14845@rox.home.comstyle.com>

23.05.2013 02:38, Brad Smith wrote:
> Fix the build of the Gtk+ UI on *BSD systems.

Can you please describe what exactly was broken on *BSD
systems, -- which file(s) needs this additional
termios.h include?

I want to fix this properly, because now it isn't
"nitpicking", as you say, anymore -- after the
previous pbonzini's changes it does not build
on debian sparc anymore due to some conflicting
includes (which is an unrelated issue, but is
triggered now because every .c file includes
a ton of #defines).

I already asked you the same question on 25.05,
which is about two weeks ago by now, -- did you
had no time to reply so far?

Thanks,

/mjt

> Signed-off-by: Brad Smith <brad@comstyle.com>
> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> Message-id: 20130521161324.GA29977@rox.home.comstyle.com
> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
> 
> diff --git a/include/qemu-common.h b/include/qemu-common.h
> index af769f5..c944bb7 100644
> --- a/include/qemu-common.h
> +++ b/include/qemu-common.h
> @@ -45,6 +45,7 @@
>  #if defined(__GLIBC__)
>  # include <pty.h>
>  #elif defined CONFIG_BSD
> +# include <termios.h>
>  # if defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__)
>  #  include <libutil.h>
>  # else
> 

      parent reply	other threads:[~2013-06-05 10:13 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130522223817.GB14845@rox.home.comstyle.com>
     [not found] ` <519E3091.2060402@msgid.tls.msk.ru>
     [not found]   ` <20130524184737.GA9642@rox.home.comstyle.com>
2013-05-25  9:14     ` [Qemu-devel] [Qemu-stable] [PATCH for 1.5] ui/gtk.c: Fix *BSD build of Gtk+ UI Michael Tokarev
2013-06-05 18:22       ` Brad Smith
2013-06-05 10:13 ` Michael Tokarev [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51AF0F46.3020002@msgid.tls.msk.ru \
    --to=mjt@tls.msk.ru \
    --cc=brad@comstyle.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.