Re: [libvirt] [PATCH 06/12] Define basic internal API for access control

2012-07-05 Thread Corey Bryant
On 05/02/2012 07:44 AM, Daniel P. Berrange wrote: From: Daniel P. Berrange berra...@redhat.com --- include/libvirt/virterror.h |3 + po/POTFILES.in|1 + src/Makefile.am | 16 + src/access/apis.txt | 577

[libvirt] [PATCH 06/12] Define basic internal API for access control

2012-05-02 Thread Daniel P. Berrange
From: Daniel P. Berrange berra...@redhat.com --- include/libvirt/virterror.h |3 + po/POTFILES.in|1 + src/Makefile.am | 16 + src/access/apis.txt | 577 + src/access/viraccessdriver.h |

Re: [libvirt] [PATCH 06/12] Define basic internal API for access control

2012-05-02 Thread Eric Blake
On 05/02/2012 05:44 AM, Daniel P. Berrange wrote: From: Daniel P. Berrange berra...@redhat.com A bit sparse on the commit message, maybe even mentioning that src/access/apis.txt contains the details would help. You are adding a new directory; do any of the cfg.mk syntax check rules need to be