confusing error message while applying push on a file
Bug #248250 reported by
Toni Ruottu
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bazaar |
Confirmed
|
Medium
|
Unassigned | ||
| Breezy |
Triaged
|
Medium
|
Unassigned | ||
Bug Description
User attempts to apply push on a file. The following happens.
$ bzr push /tmp/file
bzr: ERROR: Target directory /tmp/file already exists, but does not have a valid .bzr directory. Supply --use-existing-dir to push there anyway.
User gets confused because the error message is all about directories while he wasn't trying to do anything with any directory. Bazaar should tell the user something that the user can understand. Maybe 'Pushing to a file is not supported.' would do.
| Changed in bzr: | |
| assignee: | nobody → toni-ruottu |
| status: | New → In Progress |
Revision history for this message
| Toni Ruottu (toni-ruottu) wrote | #1 |
Revision history for this message
| James Westby (james-w) wrote | #2 |
| Changed in bzr: | |
| importance: | Undecided → Low |
| importance: | Low → Medium |
Revision history for this message
| Toni Ruottu (toni-ruottu) wrote | #3 |
| Changed in bzr: | |
| assignee: | toni-ruottu → nobody |
| Changed in bzr: | |
| status: | In Progress → Confirmed |
| tags: | added: confusing-ui push |
| tags: | added: easy |
| Changed in brz: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| tags: | added: simple |
To post a comment you must log in.
I created a new black box test that reveals the problem.
See attached patch file.