Re: psql \! WIN32 cleanup

-----Original Message-----
From: pgsql-patches-owner@postgresql.org on behalf of Tom Lane
Sent: Sat 11/6/2004 5:30 AM
To: Bruce Momjian
Cc: PostgreSQL-patches
Subject: Re: [PATCHES] psql \! WIN32 cleanup

Bruce Momjian <pgman@candle.pha.pa.us> writes:
>> The attached applied patch tests for COMSPEC in the right place and
>> supplies a reasonable shell default on Win32.
>>
> + #define DEFAULT_SHELL "c:/windows/system32/cmd.exe"

> Is it really a good idea to assume that the Windows installation is on
> drive C: ?

I think it's worse to assume the installation directory is "windows". On windows 2000 for example, the default is
"winnt".
Probably the best default would simply be "cmd.exe" as it's invariably in the system path.

Regards, Dave