Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API

2016-08-16 Thread Daniel P. Berrange
On Tue, Aug 16, 2016 at 09:21:40AM +0800, Fam Zheng wrote: > On Mon, 08/15 07:46, no-re...@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

Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API

2016-08-15 Thread Fam Zheng
On Mon, 08/15 07:46, no-re...@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

Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API

2016-08-15 Thread no-reply
Hi, Your series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Message-id: 1471270945-19975-1-git-send-email-berra...@redhat.com Subject: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based

Re: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API

2016-08-15 Thread no-reply
Hi, Your series seems to have some coding style problems. See output below for more information: Message-id: 1471270945-19975-1-git-send-email-berra...@redhat.com Subject: [Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API Type: series === TEST SCRIPT BEGIN === #!/bin/bash

[Qemu-devel] [PATCH v10 00/11] Provide a QOM-based authorization API

2016-08-15 Thread Daniel P. Berrange
This is a followup of previously posted work in 2.6 cycle: v1: https://lists.gnu.org/archive/html/qemu-devel/2016-02/msg04618.html v2: https://lists.gnu.org/archive/html/qemu-devel/2016-03/msg01454.html v3: https://lists.gnu.org/archive/html/qemu-devel/2016-03/msg02498.html v4: