[MERGE] Fix for 306394: warning, not error, on non-existent file.

Stephen J. Turnbull stephen at xemacs.org
Mon Jan 12 15:27:37 GMT 2009


Aaron Bentley writes:
 > Karl Fogel wrote:

 > >  I can do two things now:
 > > 
 > >    1) Redo the patch so the message goes to stdout and matches existing
 > >       output styles,  -OR-
 > > 
 > >    2) Redo the patch so that non-existent files are silently skipped.
 > 
 > I dislike 1, because I like the current behaviour.

I'm still confused about what there is to like about the current
behavior, except that "you shouldn't fix what can be blamed on the
user".  IMHO, a status command that errors is an out-and-out bug.

Note that programs are by-and-large stupid, and making them smart is
expensive.  I think it makes a lot of sense for bzr, which is
designed-and-implemented to track files, to politely report what it
knows ("that file doesn't exist now and was never registered in bzr")
rather than making some other program (even if it's a text editor with
pretensions of MUA-hood) duplicate that effort.




More information about the bazaar mailing list