Hi Dawid, thanks for the information. Well, please rebase to the latest commit of my master branch and resend, thanks a lot! Michal 2014-06-18 22:05 GMT+02:00 Dawid Zamirski <dzamirski@dattobackup.com>:
Ugh, now I know what happened - my local master branch has commits I have not sent pull requests for yet so it wasn't in 100% in sync with remote. Do you want me to resend the patch to the ML?
On Wed, 2014-06-18 at 16:02 -0400, Dawid Zamirski wrote:
Hi Michal,
I'm pretty sure I did git pull right before sending the patch. Here's what I did exactly:
On master branch:
git pull git checkout -b parse-param-fix origin/master created original patch & commit git format-patch -1 git send-email --no-chain-reply-to --annotate 0001-Use-long-variable-type-for-zend_parse_parameters.patch
then I've noticed the warnings (still on parse-param-fix branch) create patch & commit git fetch --all git pull --rebase git send-email --no-chain-reply-to --annotate origin/master
I guess that before starting the waring fix patch I should have create a new local branch: git checkout master git pull git checkout -b warning-fix origin/master
and then work from there.
Regards, Dawid
On Wed, 2014-06-18 at 21:28 +0200, Michal Novotny wrote:
Hi Dawid,
thanks for the patch, I'll apply it when I have time to do so. However, the patch is not critical as it's in the DPRINTF debug macro (for production environment you should disable the DEBUG macro).
Also, please make sure you are you the latest git tree (by running git pull before writing the patch) as I'm having issues applying some of the patches cleanly.
Thanks, Michal
2014-06-18 21:09 GMT+02:00 Dawid Zamirski <dzamirski@dattobackup.com>: The previous patch [1] caused compiler warnings after variable types were changed from int to long and this patch fixes this.
[1]
https://www.redhat.com/archives/libvir-list/2014-June/msg00835.html
--- src/libvirt-php.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/libvirt-php.c b/src/libvirt-php.c index 224943d..6d6fa81 100644 --- a/src/libvirt-php.c +++ b/src/libvirt-php.c @@ -3928,7 +3928,7 @@ PHP_FUNCTION(libvirt_domain_send_pointer_event) RETURN_FALSE; }
- DPRINTF("%s: x = %d, y = %d, clicked = %d, release = % d, hostname = %s...\n", PHPFUNC, pos_x, pos_y, clicked, release, hostname); + DPRINTF("%s: x = %d, y = %d, clicked = %d, release = % d, hostname = %s...\n", PHPFUNC, (int) pos_x, (int) pos_y, (int) clicked, release, hostname); ret = vnc_send_pointer_event(hostname, tmp, pos_x, pos_y, clicked, release); if (ret == 0) { DPRINTF("%s: Pointer event result is %d\n", PHPFUNC, ret); @@ -4428,7 +4428,7 @@ PHP_FUNCTION(libvirt_domain_new) numNets = i;
snprintf(tmpname, sizeof(tmpname), "%s-install", name); - DPRINTF("%s: Name is '%s', memMB is %d, maxmemMB is %d \n", PHPFUNC, tmpname, memMB, maxmemMB); + DPRINTF("%s: Name is '%s', memMB is %d, maxmemMB is %d \n", PHPFUNC, tmpname, (int) memMB, (int) maxmemMB); tmp = installation_get_xml(1, conn->conn, tmpname, memMB, maxmemMB, NULL /* arch */, NULL, vcpus, iso_image, vmDisks, numDisks, vmNetworks, numNets, @@ -6272,7 +6272,7 @@ PHP_FUNCTION(libvirt_domain_snapshot_delete) GET_SNAPSHOT_FROM_ARGS("r|l",&zsnapshot, &flags);
retval = virDomainSnapshotDelete(snapshot->snapshot, flags); - DPRINTF("%s: virDomainSnapshotDelete(%p, %d) returned %d\n", PHPFUNC, snapshot->snapshot, flags, retval); + DPRINTF("%s: virDomainSnapshotDelete(%p, %d) returned %d\n", PHPFUNC, snapshot->snapshot, (int) flags, retval); if (retval == -1) RETURN_FALSE; RETURN_TRUE; } @@ -6772,7 +6772,7 @@ PHP_FUNCTION(libvirt_storagevolume_delete) GET_VOLUME_FROM_ARGS("r|l",&zvolume,&flags);
retval = virStorageVolDelete(volume->volume, flags); - DPRINTF("%s: virStorageVolDelete(%p, %d) returned %d \n", PHPFUNC, volume->volume, flags, retval); + DPRINTF("%s: virStorageVolDelete(%p, %d) returned %d \n", PHPFUNC, volume->volume, (int) flags, retval); if (retval != 0) { set_error_if_unset("Cannot delete storage volume" TSRMLS_CC); RETURN_FALSE; -- 1.9.3
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list