We seem to use File::Temp::tempdir already elsewhere, but they are in

archimport, cvsexportcommit and cvsserver, all of which are rather rarely

used ones.  I think this is Perl 5.6.1 addition.  Is everybody Ok with
Perhaps wording this a bit more to the point?  This is triggered when

'$f' can be both a filename or a revision, so...
	File '$f' exists but it could also be the range of commits

        to produce patches for.  Please disambiguate by...
	* Saying "./$f" if you mean a file; or

        * Giving -F option if you mean a range.
Earlier I suggested that "origin^0" is a way for the user to disambiguate

favouring a rev, but such a filename can exist, so we cannot blindly

suggest to say "$f^0" here.  I think adding -F (or --format-patch) option

to send-email to explicitly disable file/directory interpretation would be

a cleaner solution for this (and it would allow you to drive this from a

script without worrying about what garbage files you happen to have in the

working tree).

--

Okay, still having --[no-]format-patch is probably a good idea indeed

for scripts. Will do.
This will break previous editor syntax hilighting stuff even more, and

has the drawback that you can't put shell sniplets in here. I think it's
Probably, I kept what was done as is, but we probably want do_edit() to

die() if the user killed it.
Will do
--=20

=C2=B7O=C2=B7  Pierre Habouzit

=C2=B7=C2=B7O                                                madcoder@debia=

n.org

OOO                                                http://www.madism.org

Thanks.

--

Ah, please scratch that "I really think" --- my mistake.
I did not check nor realize "GIT:" is what send-email already does.
--