Hi,

this is the promised runtime check for sendmail and diff.

Yours,
   Petr.

PS: I have checked that these work. This mail is from darcs send with the
patch applied.

PPS: The check is done a little too late, *after* patch selection. This should
be remedied somehow, ie. don't let people select patches if we know we won't be
able to send them. But that's for separate patch. CC-ing bugs@darcs.net.

Tue Feb  3 16:56:56 CET 2009  Petr Rockai <me@mornfall.net>
  * Cabal: Do not make absence of a diff tool fatal.

Wed Feb  4 10:13:20 CET 2009  Petr Rockai <me@mornfall.net>
  * Look for diff & sendmail at runtime, not compile time.

Attachments

• cabal_-do-not-make-absence-of-a-diff-tool-fatal_.dpatch
• unnamed

I'm just highlighting the relevant part of Petr's message in case anybody is
confused: "The check is done a little too late, *after* patch selection. Don't
let people select patches if we know we won't be able to send them."

So it seems like the logic should be: "am I sending over the network? (as
opposed to just to a file)" and if so, "can I do it?" and if not "sorry! you
don't have a working sendmail instance on your machine"

Resolved in patch505.

The following patch sent by Gabriel Kerneis <kerneis@pps.jussieu.fr> updated issue issue1344 with
status=resolved;resolvedin=2.8.0 HEAD

* resolve issue1344: abort early if can't send 
Ignore-this: 42f5d52f6e1ef0001a6f458b252e4ee4