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.
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 |