Re: Why polecat and colugos are failing to build back branches

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers(at)postgreSQL(dot)org, robert(at)logicalchaos(dot)org
Subject: Re: Why polecat and colugos are failing to build back branches
Date: 2011-06-16 01:51:59
Message-ID: 13038.1308189119@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>> As a secondary point, we have so far used mostly single quotes for
>> quoting the installation directories, in case someone wants to try other
>> funny characters besides spaces. The most recent patch uses double
>> quotes. I'm not sure what degree of support we want to achieve there.

> Oh, hm, I had noted double quotes in the one place that wasn't broken in
> test/regress/Makefile, and followed that precedent. But you're probably
> right that single quotes would be better, since that would prevent funny
> business from $ characters for instance. We're not going to be able to
> make both types of quote characters safe, unless gmake has some quoting
> function I'm not aware of (which is surely possible).

I changed the places I'd modified yesterday to use single quotes not
double, and fixed a couple other spots in passing to do the same.
However, I don't see any simple way to deal with the places where we
are sticking paths into C-string constants, to wit in building pg_config
and pg_config_paths.h. pg_config in particular is nasty because there
are pretty much guaranteed to be single quotes in the value we want for
VAL_LDFLAGS (because of rpath...). So at the moment, we're not really
safe against single *or* double quotes, nor dollar signs, in
installation paths. About the only good thing to be said about it is
that these characters are so troublesome that Unix users are unlikely
to use them in directory names anyway.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2011-06-16 01:52:02 Re: pg_upgrade using appname to lock out other users
Previous Message Stephen Frost 2011-06-16 01:33:02 Re: pg_upgrade using appname to lock out other users