'diff in a lightweight checkout' prompts for the password multiple times

Bug #430319 reported by John A Meinel on 2009-09-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Medium
Gary van der Merwe
QBzr
High
Gary van der Merwe

Bug Description

This might be a qbzr bug, but when doing: "Diff" in a lightweight checkout, it prompts me for my password 2 times. Presumably it isn't caching the connection to the branch, and thus re-opening it to get the content of both the current and target revision.

Related branches

affects: bzr-explorer → qbzr
Alexander Belchenko (bialix) wrote :

Does plain bzr diff works differently? Do you have any ideas why?

Changed in qbzr:
status: New → Incomplete

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Alexander Belchenko wrote:
> Does plain bzr diff works differently? Do you have any ideas why?
>
> ** Changed in: qbzr
> Status: New => Incomplete
>

'bzr diff' gives a single login.
'bzr qdiff' gives two logins

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkqw7bMACgkQJdeBCYSNAAM1iwCgv8mTGHnsikH3Syh0jZ3je4dx
yk4AoJJk5jBFE2d7YXUPCGoeMrNV8zEh
=/9jN
-----END PGP SIGNATURE-----

Alexander Belchenko (bialix) wrote :

I think we ought to fix it and we should think about password caching technique to help bzr-explorer problems too.

Changed in qbzr:
importance: Undecided → High
status: Incomplete → Confirmed
Gary van der Merwe (garyvdm) wrote :

To fix this, we need to change bzrlib.diff._get_trees_to_diff to also return the old and new branches.

Then bzrlib.plugins.qbzr.lib.commands.cmd_qdiff.get_diff_window_args won't need to open them separately.

Alexander Belchenko (bialix) wrote :

Gary van der Merwe пишет:
> To fix this, we need to change bzrlib.diff._get_trees_to_diff to also
> return the old and new branches.

I think you mean old and new trees, not branches?

> Then bzrlib.plugins.qbzr.lib.commands.cmd_qdiff.get_diff_window_args
> won't need to open them separately.

So, this bug is not in qbzr?

On Fri, Sep 18, 2009 at 3:50 PM, Alexander Belchenko <email address hidden> wrote:
> I think you mean old and new trees, not branches?

It currently returns the old and new trees. We need to change it to
also return the old and new branches.

> So, this bug is not in qbzr?

No, but the fix starts with a change in bzrlib.

Gary van der Merwe пишет:
> On Fri, Sep 18, 2009 at 3:50 PM, Alexander Belchenko <email address hidden> wrote:
>> I think you mean old and new trees, not branches?
>
> It currently returns the old and new trees. We need to change it to
> also return the old and new branches.

Is not tree.branch is what you need?

On Fri, Sep 18, 2009 at 4:24 PM, Alexander Belchenko <email address hidden> wrote:
> Is not tree.branch is what you need?

That is only available from WorkingTree, not RevisionTree

Gary van der Merwe пишет:
> On Fri, Sep 18, 2009 at 4:24 PM, Alexander Belchenko <email address hidden> wrote:
>> Is not tree.branch is what you need?
>
> That is only available from WorkingTree, not RevisionTree

Hmmm....

Changed in bzr:
assignee: nobody → Gary van der Merwe (garyvdm)
status: New → In Progress
importance: Undecided → Medium
Changed in bzr:
status: In Progress → Fix Committed
Gary van der Merwe (garyvdm) wrote :

Note that this will only be fixed when using bzr >= 2.1.

Changed in bzr:
status: Fix Committed → Fix Released
Changed in qbzr:
assignee: nobody → Gary van der Merwe (garyvdm)
milestone: none → 0.15
status: Confirmed → Fix Released
Changed in bzr:
milestone: none → 2.1.0b1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers