[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API |
Date: |
Tue, 16 Aug 2016 09:29:40 +0100 |
User-agent: |
Mutt/1.6.2 (2016-07-01) |
On Tue, Aug 16, 2016 at 09:21:40AM +0800, Fam Zheng wrote:
> On Mon, 08/15 07:46, address@hidden 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
Fun, those don't appear to have been used anywhere. In fact I see 4 config
options in default-configs are appear unused
$ for i in `grep -h -E '^CONFIG' default-configs/* | sed -e 's/=.*//' | sort
| uniq`
do
echo -n "$i: "
git grep $i | grep -v default-configs | wc -l
done | grep ": 0"
CONFIG_IPMI_EXTERN: 0
CONFIG_ISA_MMIO: 0
CONFIG_PAM: 0
CONFIG_PIIX_PCI: 0
so I'll send a patch to delete those
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- [Qemu-devel] [PATCH v10 07/11] util: add QAuthZ object as an authorization base class, (continued)
- [Qemu-devel] [PATCH v10 07/11] util: add QAuthZ object as an authorization base class, Daniel P. Berrange, 2016/08/15
- [Qemu-devel] [PATCH v10 06/11] qom: support arbitrary non-scalar properties with -object, Daniel P. Berrange, 2016/08/15
- [Qemu-devel] [PATCH v10 08/11] util: add QAuthZSimple object type for a simple access control list, Daniel P. Berrange, 2016/08/15
- [Qemu-devel] [PATCH v10 05/11] qapi: add a QmpInputVisitor that does string conversion, Daniel P. Berrange, 2016/08/15
- [Qemu-devel] [PATCH v10 09/11] acl: delete existing ACL implementation, Daniel P. Berrange, 2016/08/15
- [Qemu-devel] [PATCH v10 10/11] util: add QAuthZPAM object type for authorizing using PAM, Daniel P. Berrange, 2016/08/15
- [Qemu-devel] [PATCH v10 11/11] qmp: add support for mixed typed input visitor, Daniel P. Berrange, 2016/08/15
- Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API, no-reply, 2016/08/15
- Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API, no-reply, 2016/08/15