[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] seccomp: Don't include libseccomp from QEMU hea
From: |
Eduardo Otubo |
Subject: |
Re: [Qemu-devel] [PATCH] seccomp: Don't include libseccomp from QEMU header |
Date: |
Fri, 22 Sep 2017 09:54:01 +0200 |
User-agent: |
Mutt/1.8.3+47 (5f034395e53d) (2017-05-23) |
On Fri, Sep 22, 2017 at 09:09:27AM +0200, Jan Kiszka wrote:
> On 2017-09-20 10:36, Fam Zheng wrote:
> > The only prototype doesn't need anything from the lib header, and not
> > including it here allows files that include this header, for example
> > vl.c, to compile without the libseccomp cflags.
> >
> > The breakage is since c3883e1f93 for environments where `pkg-config
> > --cflags libseccomp" is non-empty.
> >
> > Reported-by: Jan Kiszka <address@hidden>
> > Signed-off-by: Fam Zheng <address@hidden>
Acked-by: Eduardo Otubo <address@hidden>
> >
> > ---
> >
> > This is an alternative for
> >
> > https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg04470.html
> > ([Qemu-devel] [PATCH] seccomp: Fix build)
> > ---
> > include/sysemu/seccomp.h | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/include/sysemu/seccomp.h b/include/sysemu/seccomp.h
> > index e67c2dc840..9b092aa23f 100644
> > --- a/include/sysemu/seccomp.h
> > +++ b/include/sysemu/seccomp.h
> > @@ -21,7 +21,5 @@
> > #define QEMU_SECCOMP_SET_SPAWN (1 << 3)
> > #define QEMU_SECCOMP_SET_RESOURCECTL (1 << 4)
> >
> > -#include <seccomp.h>
> > -
> > int seccomp_start(uint32_t seccomp_opts);
> > #endif
> >
>
> Works fine for me as well and is indeed cleaner.
>
> Jan
>
> --
> Siemens AG, Corporate Technology, CT RDA ITP SES-DE
> Corporate Competence Center Embedded Linux
--
Eduardo Otubo
Senior Software Engineer @ RedHat