qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: qemu-devel@nongnu.org
Cc: berrange@redhat.com, armbru@redhat.com, mreitz@redhat.com,
	marcandre.lureau@gmail.com, pbonzini@redhat.com,
	afaerber@suse.de
Subject: Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API
Date: Tue, 16 Aug 2016 09:21:40 +0800	[thread overview]
Message-ID: <20160816012140.GB15442@al.usersys.redhat.com> (raw)
In-Reply-To: <20160815144611.482797.13897@ex-std-node742.prod.rhcloud.com>

On Mon, 08/15 07:46, no-reply@ec2-52-6-146-230.compute-1.amazonaws.com wrote:
> /tmp/qemu-test/src/util/authz-pam.c:26:32: warning: security/pam_appl.h: No such file or directory

Hi Daniel,

I think there is an unfortunate CONFIG_ var collision you probably want to
avoid:

    $ git grep CONFIG_PAM
    configure:    echo "CONFIG_PAM=y" >> $config_host_mak
    default-configs/i386-softmmu.mak:CONFIG_PAM=y
    default-configs/x86_64-softmmu.mak:CONFIG_PAM=y
    util/Makefile.objs:$(warning $(CONFIG_PAM))
    util/Makefile.objs:util-qom-obj-$(CONFIG_PAM) += authz-pam.o

Fam

  reply	other threads:[~2016-08-16  1:28 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-15 14:22 [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 01/11] qdict: implement a qdict_crumple method for un-flattening a dict Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 02/11] option: make parse_option_bool/number non-static Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 03/11] qapi: rename QmpInputVisitor to QObjectInputVisitor Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 04/11] qapi: rename QmpOutputVisitor to QObjectOutputVisitor Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 05/11] qapi: add a QmpInputVisitor that does string conversion Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 06/11] qom: support arbitrary non-scalar properties with -object Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 07/11] util: add QAuthZ object as an authorization base class Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 08/11] util: add QAuthZSimple object type for a simple access control list Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 09/11] acl: delete existing ACL implementation Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 10/11] util: add QAuthZPAM object type for authorizing using PAM Daniel P. Berrange
2016-08-15 14:22 ` [Qemu-devel] [PATCH v10 11/11] qmp: add support for mixed typed input visitor Daniel P. Berrange
2016-08-15 14:44 ` [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API no-reply
2016-08-15 14:46 ` no-reply
2016-08-16  1:21   ` Fam Zheng [this message]
2016-08-16  8:29     ` Daniel P. Berrange

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=20160816012140.GB15442@al.usersys.redhat.com \
    --to=famz@redhat.com \
    --cc=afaerber@suse.de \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=marcandre.lureau@gmail.com \
    --cc=mreitz@redhat.com \
    --cc=pbonzini@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).