improve error message for incorrect remote diff syntax
Bug #49501 reported by
Steve Alexander
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
Confirmed
|
Wishlist
|
Unassigned |
Bug Description
When I try to diff against a remote branch with
bzr diff sftp://....
I get a bunch of misleading errors, depending on whether the location has a working tree or not. The error message I want to receive is "there is no such local file as sftp://..." That would clue me into needing to use a different syntax.
tags: | added: check-for-breezy |
To post a comment you must log in.
Confirmed. If the remote location has a working tree you get: juju/~/ dev/bzr/ jam-integration juju/~/ dev/bzr/ jam-integration /.bzr/ is not a local path.
liliana % bzr diff sftp://
bzr: ERROR: sftp://
If it does not have a working tree you get juju/srv/ bzr/public/ branches/ bzr/jam- integration juju/srv/ bzr/public/ branches/ bzr/jam- integration/ .bzr/checkout/.
liliana % bzr diff sftp://
bzr: ERROR: No WorkingTree exists for sftp://
We should clean up this sort of error handling. Though this is also part of us needing to clean up diff, which has become a very convoluted DWIM command, and really needs to be restructured to handle the different cases in a clean fashion. (It should be possible to diff against a remote branch, for example)