[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/5] ui: fix path to dbus-display1.h
From: |
Marc-André Lureau |
Subject: |
Re: [PATCH 1/5] ui: fix path to dbus-display1.h |
Date: |
Tue, 27 Sep 2022 14:38:23 +0400 |
On Tue, Sep 27, 2022 at 1:59 PM Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> While the source directory is always included in the include path,
> the corresponding directory in the build tree is not. Therefore,
> custom_targets (e.g. ui/dbus-display1.h) must be referred to using
> the full path.
>
> This avoids a build failure when ui/dbus-chardev.c is not built as
> a module:
>
> In file included from ../ui/dbus-chardev.c:32:
> ../ui/dbus.h:34:10: fatal error: dbus-display1.h: No such file or directory
> 34 | #include "dbus-display1.h"
> | ^~~~~~~~~~~~~~~~~
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> tests/qtest/dbus-display-test.c | 2 +-
> ui/dbus.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/qtest/dbus-display-test.c b/tests/qtest/dbus-display-test.c
> index 8be5974763..cb1b62d1d1 100644
> --- a/tests/qtest/dbus-display-test.c
> +++ b/tests/qtest/dbus-display-test.c
> @@ -3,7 +3,7 @@
> #include <gio/gio.h>
> #include <gio/gunixfdlist.h>
> #include "libqtest.h"
> -#include "dbus-display1.h"
> +#include "ui/dbus-display1.h"
>
> static GDBusConnection*
> test_dbus_p2p_from_fd(int fd)
> diff --git a/ui/dbus.h b/ui/dbus.h
> index c001c11f70..9c149e7b41 100644
> --- a/ui/dbus.h
> +++ b/ui/dbus.h
> @@ -31,7 +31,7 @@
> #include "ui/console.h"
> #include "ui/clipboard.h"
>
> -#include "dbus-display1.h"
> +#include "ui/dbus-display1.h"
>
> typedef struct DBusClipboardRequest {
> GDBusMethodInvocation *invocation;
> --
> 2.37.3
>
- [PATCH 0/5] Small meson updates for QEMU 7.2, Paolo Bonzini, 2022/09/27
- [PATCH 1/5] ui: fix path to dbus-display1.h, Paolo Bonzini, 2022/09/27
- Re: [PATCH 1/5] ui: fix path to dbus-display1.h,
Marc-André Lureau <=
- [PATCH 2/5] meson: require 0.61.3, Paolo Bonzini, 2022/09/27
- [PATCH 3/5] meson: multiple names can be passed to dependency(), Paolo Bonzini, 2022/09/27
- [PATCH 4/5] configure, meson: move C++ compiler detection to meson.build, Paolo Bonzini, 2022/09/27