Hi,

apply --reply is mostly broken. If a patch fails (e.g. because of a
conflict), function "sendit" in "redirectOutput" concatenates the error
message to the output file name instead of writing the message to the file:
               sendit tempf e =
                do sendSanitizedEmail opts f to "Darcs error applying
patch!" cc $
                             tempf ++ "\n\nCaught exception:\n"++
                             show e++"\n"
                   throwIO ExitSuccess

If a patch is applied successfully, no email is sent, because "sendit"
is only called from the exception handler. It looks like
"redirectOutput" expects "applyItNow" to call "exitWith ExitSuccess" at
the end.

Ilya

Any chance you could help us with a test case?
http://wiki.darcs.net/Development/RegressionTests

Hey, thanks for following up with tests, Ilya!  I wonder where this 
regressed (ie. if we can track down the darcs version where it was last 
known working).  Would be a bit reassuring to find one, but seems just 
as broken in darcs 2.5 (I had to tweak the test because --patch-name was 
renamed to --name due to issue1883, sigh)

Looks like Ilya has provided a diagnosis, so I think we just need a 
patch now.

This should be fixed in 2.10. I have tried to find out why apply does
not send any mail but without success so far.