Re: [COMMITTERS] pgsql: Have dtrace depend on object files directly, not objfiles.txt

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Have dtrace depend on object files directly, not objfiles.txt
Date: 2015-10-15 17:16:47
Message-ID: CA+TgmoYaYT8AVsjJVgF6MiZ51KUqFsxiUG+aoWYdAVNFqLvnOg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, Oct 15, 2015 at 1:01 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <rhaas(at)postgresql(dot)org> writes:
>> Have dtrace depend on object files directly, not objfiles.txt
>
> This patch appears to have broken parallel builds. I get this:
>
> $ make -j8 -s
> cat: access/objfiles.txt: No such file or directory
> cat: bootstrap/objfiles.txt: No such file or directory
> cat: catalog/objfiles.txt: No such file or directory
> cat: parser/objfiles.txt: No such file or directory
> cat: commands/objfiles.txt: No such file or directory
> cat: executor/objfiles.txt: No such file or directory
> ... lots more omitted ...
> cat: storage/objfiles.txt: No such file or directory
> cat: tcop/objfiles.txt: No such file or directory
> cat: tsearch/objfiles.txt: No such file or directory
> cat: utils/objfiles.txt: No such file or directory
> In file included from gram.y:14919:
> scan.c: In function 'yy_try_NUL_trans':
> scan.c:10307: warning: unused variable 'yyg'
> All of PostgreSQL successfully made. Ready to install.
>
> and this:
>
> $ make -s clean
> cat: ../../src/timezone/objfiles.txt: No such file or directory
>
> despite the fact that I'm not using dtrace. I imagine this
> comes from gmake trying to evaluate the dependencies for
> utils/probes.o before all the subdirectories have been built.
>
> Please fix or revert.

Thanks for the report. I have reverted the patch.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2015-10-15 17:23:23 Re: [COMMITTERS] pgsql: Have dtrace depend on object files directly, not objfiles.txt
Previous Message Robert Haas 2015-10-15 17:16:23 pgsql: Revert "Have dtrace depend on object files directly, not objfile

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2015-10-15 17:18:35 Re: DTrace build dependency rules
Previous Message Robert Haas 2015-10-15 17:14:39 Re: Foreign join pushdown vs EvalPlanQual