Re: Re: pgsql: Cosmetic improvements in plpython's make rule for libpython impo

From: Noah Misch <noah(at)leadboat(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: Re: pgsql: Cosmetic improvements in plpython's make rule for libpython impo
Date: 2014-08-17 16:42:18
Message-ID: 20140817164218.GA410471@tornado.leadboat.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Sun, Aug 17, 2014 at 11:55:09AM -0400, Tom Lane wrote:
> Noah Misch <noah(at)leadboat(dot)com> writes:
> > On Fri, Feb 14, 2014 at 04:31:42PM +0000, Tom Lane wrote:
> >> Cosmetic improvements in plpython's make rule for libpython import library.
> >>
> >> This build technique is remarkably ugly, but that doesn't mean it has
> >> to be unreadable too. Be a bit more liberal with the vertical whitespace,
> >> and give the .def file a proper dependency, just in case.
>
> > That dependency omission was no accident. See commit 0d147e43 and
> > contemporary discussion.
>
> Hm. I failed to find the "contemporary discussion" after a bit of
> looking,

http://www.postgresql.org/message-id/flat/20130103031358(dot)GB11705(at)tornado(dot)leadboat(dot)com

> but surely there's a way to get Cygwin's make to handle file
> names with colons in them? Or if not, maybe we should try to strip out
> any drive letter appearing in the DLL spec?

Escaping the colon with a backslash does work. (There's nothing
Cygwin-specific in play, just standard GNU make colon semantics.)

> I do not really believe the
> argument that we don't need a dependency there.

We don't add a dependency on /usr/lib/libz.a, even though a change to that
file might influence a rebuild. This is similar.

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2014-08-17 19:59:16 pgsql: Improve DISCARD documentation.
Previous Message Tom Lane 2014-08-17 15:55:09 Re: Re: pgsql: Cosmetic improvements in plpython's make rule for libpython impo