Re: Re: pgsql: Cosmetic improvements in plpython's make rule for libpython impo
От | Noah Misch |
---|---|
Тема | Re: Re: pgsql: Cosmetic improvements in plpython's make rule for libpython impo |
Дата | |
Msg-id | 20140817164218.GA410471@tornado.leadboat.com обсуждение исходный текст |
Ответ на | Re: Re: pgsql: Cosmetic improvements in plpython's make rule for libpython impo (Tom Lane <tgl@sss.pgh.pa.us>) |
Список | pgsql-committers |
On Sun, Aug 17, 2014 at 11:55:09AM -0400, Tom Lane wrote: > Noah Misch <noah@leadboat.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.GB11705@tornado.leadboat.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.
В списке pgsql-committers по дате отправления: