Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Memcached-libmemcached CPAN distribution.

Report information
The Basics
Id: 102858
Status: resolved
Priority: 0/
Queue: Memcached-libmemcached

People
Owner: Nobody in particular
Requestors: zefram [...] fysh.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.001801



Subject: lib build is parallel
Date: Wed, 18 Mar 2015 11:27:33 +0000
To: bug-Memcached-libmemcached [...] rt.cpan.org
From: Zefram <zefram [...] fysh.org>
libmemcached by default builds itself in a heavily parallel fashion. In the context of a CPAN module build, this isn't ideal: CPAN module builds are generally serial internally, so it makes more sense to arrange parallelism at a higher level, and when that's done a single internally-parallel module build results in more parallelism than is desired. I think the CPAN module should tell the C library to build itself serially (which it supports via configure option). The attached patch implements the serialisation, in an unconditional manner. If you believe the parallel building to be of significant value in the CPAN context, I'd also be totally satisfied if the choice between serial and parallel building were controlled by a Makefile.PL command-line option or an environment variable. -zefram

Message body is not shown because sender requested not to inline it.

Sounds good. Thanks Zefram!
I should note that the code is on github and we're using that issue tracker. I've opened a case there: https://github.com/timbunce/Memcached-libmemcached/issues/17 A pull request would be delightful!
Subject: Re: [rt.cpan.org #102858] lib build is parallel
Date: Wed, 18 Mar 2015 14:06:34 +0000
To: Tim_Bunce via RT <bug-Memcached-libmemcached [...] rt.cpan.org>
From: Zefram <zefram [...] fysh.org>
Tim_Bunce via RT wrote: Show quoted text
>A pull request would be delightful!
Sorry, I don't drink the github kool-aid. -zefram
On 2015-03-18 10:06:48, zefram@fysh.org wrote: Show quoted text
> Tim_Bunce via RT wrote:
> >A pull request would be delightful!
> > Sorry, I don't drink the github kool-aid.
But you could at least create your patches with git-format-patch, so the commit message + author line is already there.
Zefram's patch is applied in 1.001801