From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Improve speed of make check-world |
Date: | 2015-05-12 15:01:49 |
Message-ID: | 20150512150149.GJ385@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On Sat, May 9, 2015 at 02:02:15PM -0400, Bruce Momjian wrote:
> Yes, I found this odd too. It is caused by this commit:
>
> commit dcae5faccab64776376d354decda0017c648bb53
> Author: Peter Eisentraut <peter_e(at)gmx(dot)net>
> Date: Thu Apr 23 08:59:52 2015 -0400
>
> Improve speed of make check-world
>
> It basically added a'check' target dependency of 'temp-install' to
> Makefile.global.in, which added it to every Makefile containing a
> 'check' target.
>
> While you can't easily add rule action to an existing target, you can
> easily add dependencies by just mentioning the target multiple times,
> e.g.
>
> check: dep1
>
> check: dep2
>
> dep1:
> echo 1
>
> dep2:
> echo 2
>
>
> Running 'make check' will output '1 2'.
>
> Now, of course many check's need temp-install, but the SGML build does
> not. The attached diff uses a symbol defined in doc/src/sgml/Makefile
> (GENERATED_SGML) to supress the temp-install target rule action.
Patch applied, with added Makefile comment.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ Everyone has their own god. +
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2015-05-12 15:12:41 | pgsql: "Fix" test_ddl_deparse regress test schedule |
Previous Message | Bruce Momjian | 2015-05-12 15:01:32 | pgsql: doc: prevent SGML 'make check' from building temp install |