From: | Martijn van Oosterhout <kleptog(at)svana(dot)org> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Thomas Hallgren <thhal(at)mailblocks(dot)com>, Dave Page <dpage(at)vale-housing(dot)co(dot)uk>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: pg_config --pgxs on Win32 |
Date: | 2005-10-13 19:08:55 |
Message-ID: | 20051013190855.GC6080@svana.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Oct 13, 2005 at 02:53:09PM -0400, Tom Lane wrote:
> Thomas Hallgren <thhal(at)mailblocks(dot)com> writes:
> > I do have a workaround in place that makes it work for me now. I do
> > $(dir $(subst \\,/,xxx)) and that works fine but given that the targeted
> > platform for pgxs on Win32 is MinGW, perhaps it should output forward
> > slashes anyway.
>
> I've already applied that change. Your gripe was pretty much exactly
> what I was afraid we'd hear, namely that the double-backslash output
> would break cases that used to work. I think we should stick with the
> forward slash output, since that's what we know has been working in the
> 8.0 branch.
Besides, Windows has accepted the forward slash as directory seperator
all the way since DOS 2.0 when directories were introduced. The only
reason they sometimes don't work as expected is because it's also the
option indicator :(.
Presumably things break when commands read /path as an option rather
than a path. Hopefully they're smart enough to realize c:/path does not
actually contain an option but is just a path. Presumably that's
fixable some other way?
--
Martijn van Oosterhout <kleptog(at)svana(dot)org> http://svana.org/kleptog/
> Patent. n. Genius is 5% inspiration and 95% perspiration. A patent is a
> tool for doing 5% of the work and then sitting around waiting for someone
> else to do the other 95% so you can sue them.
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2005-10-13 19:09:58 | Re: auto vacuum lock on 8.1beta1 |
Previous Message | Tom Lane | 2005-10-13 18:53:09 | Re: pg_config --pgxs on Win32 |