[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/9] util/qemu-openpty.c: Don't assume pty.h is glibc-only
From: |
Thomas Huth |
Subject: |
Re: [PATCH v3 2/9] util/qemu-openpty.c: Don't assume pty.h is glibc-only |
Date: |
Fri, 3 Jul 2020 17:12:28 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 03/07/2020 16.56, Peter Maydell wrote:
> From: David CARLIER <devnexen@gmail.com>
>
> Instead of using an OS-specific ifdef test to select the "openpty()
> is in pty.h" codepath, make configure check for the existence of
> the header and use the new CONFIG_PTY instead.
>
> This is necessary to build on Haiku, which also provides openpty()
> via pty.h.
>
> Signed-off-by: David Carlier <devnexen@gmail.com>
> [PMM: Expanded commit message]
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> configure | 9 +++++++++
> util/qemu-openpty.c | 2 +-
> 2 files changed, 10 insertions(+), 1 deletion(-)
[...]
> diff --git a/util/qemu-openpty.c b/util/qemu-openpty.c
> index 2e8b43bdf57..9d8ad6905e0 100644
> --- a/util/qemu-openpty.c
> +++ b/util/qemu-openpty.c
> @@ -35,7 +35,7 @@
> #include "qemu/osdep.h"
> #include "qemu-common.h"
>
> -#if defined(__GLIBC__)
> +#if defined CONFIG_PTY
Shouldn't there be some parentheses around CONFIG_PTY here?
Thomas