[PATCH] http-push: fix missing "#ifdef USE_CURL_MULTI" around "is_running_queue"

[PATCH] http-push: fix missing "#ifdef USE_CURL_MULTI" around "is_running_queue"

[Christian Couder]

As it is breaking the build when USE_CURL_MULTI is not defined.

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
---
 http-push.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

This patch applies to pu.

diff --git a/http-push.c b/http-push.c
index ef1a9f3..14f3996 100644
--- a/http-push.c
+++ b/http-push.c
@@ -2291,7 +2291,9 @@ int main(int argc, char **argv)
 		repo->url = rewritten_url;
 	}
 
+#ifdef USE_CURL_MULTI
 	is_running_queue = 0;
+#endif
 
 	/* Verify DAV compliance/lock support */
 	if (!locking_available()) {
-- 
1.6.3.rc1.112.g17e25

Re: [PATCH] http-push: fix missing "#ifdef USE_CURL_MULTI" around "is_running_queue"

[Tay Ray Chuan]

Hi,

On Mon, May 11, 2009 at 12:44 AM, Christian Couder
<chriscool@tuxfamily.org> wrote:
> As it is breaking the build when USE_CURL_MULTI is not defined.

thanks for catching this. By the way, how did you catch this? Are you
building without USE_CURL_MULTI defined?

-- 
Cheers,
Ray Chuan

Re: [PATCH] http-push: fix missing "#ifdef USE_CURL_MULTI" around "is_running_queue"

[Christian Couder]

Hi,

Le lundi 11 mai 2009, Tay Ray Chuan a écrit :
> Hi,
>
> On Mon, May 11, 2009 at 12:44 AM, Christian Couder
>
> <chriscool@tuxfamily.org> wrote:
> > As it is breaking the build when USE_CURL_MULTI is not defined.
>
> thanks for catching this. By the way, how did you catch this? Are you
> building without USE_CURL_MULTI defined?

My main development box still runs an old Kubuntu Feisty (7.04). It
has some libcurl3 packages version 7.15.5-1ubuntu2.

In "/usr/include/curl/curlver.h", I have:

#define LIBCURL_VERSION_NUM 0x070f05

and in "http.h" there is:

#if LIBCURL_VERSION_NUM >= 0x071000
#define USE_CURL_MULTI
#define DEFAULT_MAX_REQUESTS 5
#endif

so yes, I think USE_CURL_MULTI is not defined on my box and that's why
the build failed on it.

Regards,
Christian.