qbranch should determine valid destination

Bug #318214 reported by Alexander Belchenko
2
Affects Status Importance Assigned to Milestone
QBzr
Fix Released
Medium
Ian Clatworthy

Bug Description

If user don't provide valid destination in qbranch dialog then operation fails with message:

Target directory "" already exists.

We should be more smart and auto-fill the destination based on the source URL (as plain bzr branch does).

Related branches

Changed in qbzr:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Mark Hammond (mhammond) wrote :

Note that 'qgetnew' is smart about filling the destination, but I didn't apply the same thing to qbranch as I didn't get any reply to my mail asking if it was desirable. I think it is, so refactoring the qgetnew code into something reusable might make sense.

Revision history for this message
Alexander Belchenko (bialix) wrote : Re: [Bug 318214] Re: qbranch should determine valid destination

Mark Hammond пишет:
> Note that 'qgetnew' is smart about filling the destination, but I didn't
> apply the same thing to qbranch as I didn't get any reply to my mail
> asking if it was desirable. I think it is, so refactoring the qgetnew
> code into something reusable might make sense.
>
Sorry, it seems like I've missed your mail.

Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

When the user enters a source location, the destination location should change to reflect it, provided the source isn't relative. For example:

1. source=lp:qbzr should suggest qbzr
2. source=trunk should suggest nothing.

As a nice bit of dwim, source=lp:bzr-xxx should suggest xxx.

I'd really like to see this done soon. See https://lists.launchpad.net/bzr-doc/msg00005.html.

Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

Branch ready for review

Changed in qbzr:
status: Confirmed → Fix Committed
assignee: nobody → Ian Clatworthy (ian-clatworthy)
Changed in qbzr:
milestone: none → 0.17
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.