Joey Hess jo...@debian.org writes:
I think this should only be done if DEB_BUILD_OPTIONS has parallel=N,
and should limit dpkg-deb processes to the N. If someone sends a patch
doing that, I'll apply it.
Such a patch (against git) is attached below. I did little testing only:
it doesn't break
Gergely Nagy alger...@madhouse-project.org writes:
Joey Hess jo...@debian.org writes:
I think this should only be done if DEB_BUILD_OPTIONS has parallel=N,
and should limit dpkg-deb processes to the N. If someone sends a patch
doing that, I'll apply it.
Such a patch (against git) is
Gergely Nagy wrote:
+my $processes=0;
+my $max_procs=0;
Wouldn't it be simpler to default $max_procs to 1, rather than the
current special-cased handling of 0 via $forked etc?
+ } else {
Hmm, there is no check for fork having failed. ($pid would be undef)
+ if (++$processes
Joey Hess jo...@debian.org writes:
Gergely Nagy wrote:
+my $processes=0;
+my $max_procs=0;
Wouldn't it be simpler to default $max_procs to 1, rather than the
current special-cased handling of 0 via $forked etc?
Good point, it would be.
+} else {
Hmm, there is no check for fork
Gergely Nagy alger...@madhouse-project.org writes:
I'll add fork() error handling and the above change, retest send an
updated patch.
Updated patch attached below. As you suggested, $processes and
$max_procs are initialised to 1 now, and the process waiting was
reworked.
Waiting in the else
Kari Pahula wrote:
I did some tests on this, making a rather trivial change to
dh_builddeb's foreach loop, like this:
my $processes = 0;
foreach ... {
my $pid = fork();
if ($pid == 0) {
...
exit 0;
} else {
if (++$processes 1) {
wait;
Package: debhelper
Version: 7.9.3
Severity: wishlist
It'd be useful to have dh_builddeb have support for building debs in
parallel. As a case, the ghc6 6.12.3 package I'm working on now has
debs sized 4M, 8M, 16M, 32M and 43M. After building it all I'm fairly
sure to have it all in memory and
7 matches
Mail list logo
