Re: [COMMITTERS] pgsql: Improved parallel make support

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Improved parallel make support
Date: 2010-11-13 21:48:10
Message-ID: 22745.1289684890@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> Well, it looks like $(eval) is pretty broken in 3.80, so either we
> require 3.81 or we abandon this line of thought.

[ emerges from some grubbing about in the gmake sources... ]
It looks to me like the bug in 3.80 is only triggered when "eval"
expands to a long enough string to trigger reallocation of the variable
buffer. (Ergo, the reason they didn't find it sooner was they only
tested on relatively short strings.)

I wonder whether the bug could be worked around if you did the iteration
on SUBDIRS in a foreach surrounding the eval call, so that each eval
dealt with only one subdir target. This would result in a bit of
redundancy in the generated rules, but that seems tolerable.

regards, tom lane

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Dave Page 2010-11-14 10:30:59 Re: [COMMITTERS] pgsql: Improved parallel make support
Previous Message Peter Eisentraut 2010-11-13 20:13:17 Re: [COMMITTERS] pgsql: Improved parallel make support

Browse pgsql-hackers by date

  From Date Subject
Next Message Stefan Kaltenbrunner 2010-11-13 21:53:52 Re: 8.4-vintage problem in postmaster.c
Previous Message Kevin Grittner 2010-11-13 20:15:40 SSI update