[darcs-devel] ask-deps quit

[Stephen J. Turnbull]

Eric Y. Kow writes:

 > Hi Kevin/Tommy/Stephen,
 > 
 > Stephen: Quick clarification question about a bug you filed in 2005
 > (quitting from --ask-deps records the empty patch anyway)
 >   http://bugs.darcs.net/issue308
 > 
 > > The only inconsistency is that if both recordings and dependencies are
 > > empty darcs will not abort but create a noop patch; it's an awkward
 > > feature.
 > 
 > Is this what you meant by 'records the empty patch anyway'?

Yes, I think so.  Empty patches have their uses, but I would prefer
that "q" quit here.  It could either be hardcoded to prompt whether to
really create the empty patch (this is a rare use case either way, I
think), or there could be a --record-empty-patch flag with values
"never/no", "ask", "always/yes".  This would allow people to set
defaults in preferences.  That may be overkill though.