[darcs-users] darcs patch: Cabal: Look around for diff and sendmail properly.

[Petr Rockai]

Eric Kow <kowey at darcs.net> writes:
> This falls back to "false" (/bin/false on my machine) instead of setting
> SENDMAIL to the empty string.

> Is this behaviour more desirable than the current behaviour of setting
> it to the empty string?  Are we relying on the current behaviour in any
> way (the current code uses the empty string, as far as I can tell)?
Don't really know, but it shouldn't matter either way. I just thought that
runProcess "" isn't particularly well defined, so I put in something every
POSIX system should have (we don't use the interface on win32).

Yours,
   Petr.

-- 
Peter Rockai | me()mornfall!net | prockai()redhat!com
 http://blog.mornfall.net | http://web.mornfall.net

"In My Egotistical Opinion, most people's C programs should be
 indented six feet downward and covered with dirt."
     -- Blair P. Houghton on the subject of C program indentation