Re: [PATCH spice-gtk] build-sys: fix 'Only <spice-client.h> can be included directly' warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 15, 2016 at 12:39 PM, Marc-André Lureau
<marcandre.lureau@xxxxxxxxx> wrote:
> Tests should be allowed to include directly private headers. Set
> __SPICE_CLIENT_H_INSIDE__ where necessary or include spice-client.h
> where possible.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
> ---
>  tests/session.c | 4 +---
>  tests/util.c    | 1 +
>  2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/tests/session.c b/tests/session.c
> index feee3db..d065b60 100644
> --- a/tests/session.c
> +++ b/tests/session.c
> @@ -1,6 +1,4 @@
> -#include <glib.h>
> -
> -#include "spice-session.h"
> +#include <spice-client.h>
>
>  static void test_session_uri(void)
>  {
> diff --git a/tests/util.c b/tests/util.c
> index b9b9535..dcc9770 100644
> --- a/tests/util.c
> +++ b/tests/util.c
> @@ -3,6 +3,7 @@
>  #include <string.h>
>  #include <stdlib.h>
>
> +#define __SPICE_CLIENT_H_INSIDE__
>  #include "spice-util-priv.h"
>
>  enum {
> --
> 2.5.0
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Acked-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]