Re: [libvirt-php PATCH 0/2] Add binding for virDomainInterfaceAddresses

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michal,

Yes, please feel free to add:

Signed-off-by: Dawid Zamirski <dzamirski@xxxxxxxxx>

to both messages. I'll make sure it is included in my future
submissions from now on.

Thanks,
Dawid

On Fri, 2019-07-12 at 17:12 +0200, Michal Privoznik wrote:
> On 7/8/19 11:32 PM, Dawid Zamirski wrote:
> > Hello,
> > 
> > The following two patches add a new bingding for
> > virDomainInterfaceAddresses.
> > While working on it I have found that the PHP7 version of the
> > VIRT_ARRAY_INIT macro was causing segfaults which I have fixed in
> > the
> > first patch wheread the actual implementation of the binding is in
> > the
> > second one. Details are included in the commit messages of each
> > patch.
> > 
> > Regards,
> > Dawid
> > 
> > Dawid Zamirski (2):
> >    Fix PHP7 VIRT_ARRAY_INIT macro implementation.
> >    Add binding for virDomainInterfaceAddresses.
> > 
> >   src/libvirt-domain.c | 55
> > ++++++++++++++++++++++++++++++++++++++++++++
> >   src/libvirt-domain.h |  2 ++
> >   src/libvirt-php.c    |  9 ++++++++
> >   src/util.h           |  7 +++---
> >   4 files changed, 70 insertions(+), 3 deletions(-)
> > 
> 
> Patches look good, but I cannot push them because they are missing
> your 
> signoff. I can fix it, but I need your permisson to add the following
> at 
> the end of both messages:
> 
>    Signed-off-by: Dawid Zamirski <dzamirski@xxxxxxxxx>
> 
> This confirms also that you've agreed to Developer Certificate of
> Origin 
> 1.1:
> 
>    https://libvirt.org/hacking.html
> 
> Michal
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux