attempt to check out SVN repo never gets anywhere

Bug #418252 reported by Stuart P. Bentley
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Bazaar Subversion Plugin
Invalid
Undecided
Unassigned
QBzr
Fix Released
High
Gary van der Merwe
TortoiseBZR
Invalid
Undecided
Unassigned

Bug Description

When I try to check out an SVN branch like http://tortoisesvn.tigris.org/svn/tortoisesvn/trunk/src/Resources/svg/Overlays/ (I was doing this one as a branch copy), tbzrcommandw starts bzr, which subsequently does nothing (as far as I can tell from the blank status box and progress bar). If I hit cancel, nothing changes until I manually kill the bzr process via Task Manager.

About Tortoise Bazaar
Tortoise Bazaar 0.2rc1
Executable: C:\Program Files\Bazaar\tbzrcommandw.exe
Library: C:\Program Files\Bazaar\lib\library.zip\tbzrlib
Copyright 2008-2009 Canonical Ltd.
About Bazaar
Bazaar (bzr) 1.18rc1
  Python interpreter: C:\Program Files\Bazaar\python25.dll 2.5.4
  Python standard library: C:\Program Files\Bazaar\lib\library.zip
  bzrlib: C:\Program Files\Bazaar\lib\library.zip\bzrlib
  Bazaar configuration: C:\Documents and Settings\Stuart P. Bentley\Application Data\bazaar\2.0
  Bazaar log file: C:\Documents and Settings\Stuart P. Bentley\My Documents\.bzr.log

Copyright 2005, 2006, 2007, 2008, 2009 Canonical Ltd.
http://bazaar-vcs.org/

bzr comes with ABSOLUTELY NO WARRANTY. bzr is free software, and
you may use, modify and redistribute it under the terms of the GNU
General Public License version 2 or later.

About Plugins
bzrtools 1.18
Various useful commands for working with bzr.
launchpad 1.18rc1
Launchpad.net integration plugin for Bazaar.
netrc_credential_store 1.18rc1
Use ~/.netrc as a credential store for authentication.conf.
qbzr 0.13.1
QBzr - Qt-based frontend for Bazaar
rebase 0.5.3
Rebase support.
svn 0.6.4
Support for Subversion branches

Revision history for this message
Stuart P. Bentley (stuart) wrote :

Doing this from the command shell successfully leads me to believe that this bug is caused by bzr prompting for the username and password.

Revision history for this message
Alexander Belchenko (bialix) wrote :

So, actually this bug in QBzr dialogs (because TortoiseBzr uses QBzr dialogs under the hood).

But the root of the problem begins from bzr-svn plugin: it should use usual bzrlib UIFactory methods to obtain password. I suspect it now uses direct access to console or something similar.

Revision history for this message
Alexander Belchenko (bialix) wrote :

I'd like to get some comments about this bug from main bzr-svn author Jelmer Vernoij before deciding what to do about it in QBzr.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

bzr-svn doesn't prompt for passwords itself but utilizes the UI factory.

Changed in bzr-svn:
status: New → Incomplete
Revision history for this message
Alexander Belchenko (bialix) wrote : Re: [Bug 418252] Re: attempt to check out SVN repo never gets anywhere

Jelmer Vernooij пишет:
> bzr-svn doesn't prompt for passwords itself but utilizes the UI factory.

We have support in QBzr to handle password prompt for transports, and it
works OK for other transports (e.g. bzr+ssh, ftp). Something different
with bzr-svn then.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Alexander Belchenko wrote:
> Jelmer Vernooij пишет:
>> bzr-svn doesn't prompt for passwords itself but utilizes the UI factory.
>
> We have support in QBzr to handle password prompt for transports, and it
> works OK for other transports (e.g. bzr+ssh, ftp). Something different
> with bzr-svn then.
>
On what platforms does this problem appear?

Jelmer
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iQIcBAEBCAAGBQJKm4n7AAoJEACAbyvXKaRXO78P/Rv2aywVOuC1TgCYxGEgPTgX
AKJw+WZEhiySB37Eb7fATHJ6CmnUSNOMwDc99UWxRa73D2EdNmAAcxA4KUg3jbjx
kzG1g1rYUnGujF83JbOqRDjwdckdCMlPENSCzB1QGNuYPX/1ticj3/OpplLUwOmV
HMhDbTiaRYoB8Eg+29S7ol+pnGmuO79MxjqQzeaO9bLDIs0jMmJ2xSzUE2721VU+
iDMalQxyRSTFvnoe6U9ujKxwP3V4MRUAHkNMAW1FpbvhdovsXQV0N9mqMN2+9DGe
57bKhJ3eYwpwUI1ZHJai0cWtXSC9Q+pvlfJ0BT9nFWPoq/Zh8vSZDjDFGWeYO2ka
cOLCQ++qT/lGiAkgyILGYVH5VoWT7s1y5dv7GTsgJMKpYQpfhRQZXIC1udRcbo41
rtVRCoVp1TMKa4auwZsJHCveDMWeoKkBVPBJtmaSDAy4L8SjpiHQaWrjEyO70r1c
zAiWN6m8x8xuj1Xz2O7i9fybZWqFjZqE/ge8GLsXh3LX/lS5J+xZBivISuqWgRY1
snvZNFFHfefU613WVFPd2JOepEdp0YctcBYYpyDe35MyGwXggyN1rX1PDFPmvx57
oW7QlZKM/h43yBnclonu6FQnkhGYhyj6RpTcylwgEnSZiy0wTrmbCeJDTXLKkFvO
f46zYXEyulUFX5RNqBkw
=TKny
-----END PGP SIGNATURE-----

Revision history for this message
Alexander Belchenko (bialix) wrote :

Originally bug filed as problem with TortoiseBzr. TortoiseBzr works only on Windows.

Revision history for this message
Alexander Belchenko (bialix) wrote :

Jelmer Vernooij пишет:
> Alexander Belchenko wrote:
>> Jelmer Vernooij пишет:
>>> bzr-svn doesn't prompt for passwords itself but utilizes the UI factory.
>> We have support in QBzr to handle password prompt for transports, and it
>> works OK for other transports (e.g. bzr+ssh, ftp). Something different
>> with bzr-svn then.
>>
> On what platforms does this problem appear?

Jelmer, I'm sure it does not depend on platform.
Can you point me exact code in bzr-svn which get password via UI factory?

Revision history for this message
Jelmer Vernooij (jelmer) wrote : Re: About bug 418252

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

Hi Alexander,

Alexander Belchenko wrote:
> can you help us with bug 418252
> (https://bugs.launchpad.net/tortoisebzr/+bug/418252). I've asked
> you about the code in bzr-svn which used to get password. I need to
> know which ui methods you're using.
On Windows we use a Windows-specific credentials provider that is
provided by the Subversion libraries; I'm not sure what that does
exactly. On unix, we only rely on AuthenticationConfig.get_user() and
AuthenticationConfig.get_password().

Cheers,

Jelmer
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10rc1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQIcBAEBAgAGBQJKnlz8AAoJEACAbyvXKaRXDmwP/0reIdV3SXxmwb0SDrBBPosH
xcY8pA5dHIpKXZ7t+ZVTvgMFUI0xAyTiJc+H/O1fqKXsK54+RNY1BrdY4n0iBhfV
qvublmAjG0vA9yfZYnyJL06rezkr8P/PuGOEw4F3QYm36soYy4cZSIguFeeJFXNt
Thi5ue8XZ48ULA7NStswIyaKpEHs7HzQXhIR9r+mc50mkKFXfAm1SMUD8KGwFZk/
DYmJlPPgoCLduXBf57lE6/PwahZeklaNan/8ykz1Z8zy25qmaoIit6X2L9rjpZ0q
DCMbSy1qfbZeGcAwRiFoOvLHEBM0G1uU23oXTQx4BJcnq10AHg2B7FYhA355Rzb5
sMEArG7jB2wFjExFJ42bmZ0AVKVXzUytACZ+1G+81Qid93gTBpP57VuqPhqy9IsZ
F50UrZa2ou9lZsXpfcKeyRZURfIiiaTNQRNUtJRGqJ61yOEgIT7dFm5/NSniSWs3
9YUmzEvSkWm8pWweWGLC5QdDvyzxzXSFa+WYtcgrg0/v4OohlB8xcuc8QZP3aHfH
HEVHDeRcwpY6PoR16Hf2Eyahsw7er/hfkci8/pk1htjXTkHQ5LOXqSn5eYCV1FPy
8EQ6mBl5MuaEHigkNtDXjxWvalRR7uMCQ0phM4hRtPXHV+zwe/NQMhvAOowXPwWK
Yu45hfUgcfKvrV27y2zj
=9fKx
-----END PGP SIGNATURE-----

Revision history for this message
Alexander Belchenko (bialix) wrote :

So, QBzr cannot help here, because bzr-svn does not use ui.get_password method at all. And because TBZR uses QBzr under the hood, this means you can't access password protected svn repo on Windows with TortoiseBzr.

Changed in qbzr:
status: New → Invalid
Revision history for this message
Jelmer Vernooij (jelmer) wrote : Re: [Bug 418252] Re: attempt to check out SVN repo never gets anywhere

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

Alexander Belchenko wrote:
> So, QBzr cannot help here, because bzr-svn does not use
> ui.get_password method at all. And because TBZR uses QBzr under the
> hood, this means you can't access password protected svn repo on
> Windows with TortoiseBzr.
It *does* use ui.get_password(), just not directly. bzr-svn uses
bzrlib.config.AuthenticationConfig.get_{user,password} and *those* two
functions use bzrlib's ui interface.

I don't see why this bug report should be considered invalid.

Cheers,

Jelmer
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10rc1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQIcBAEBAgAGBQJKnmD9AAoJEACAbyvXKaRXE2IP/j4PzsZJ4DRjnWsJVyhYZ4Or
ELLUmAp2dBdrSFN8imJ6jQZ3OuWyUV53D1Bc55k/3DFp1lEHtwaNMkZU9j4JxKRW
AC/uB3hKhBjcnRieIreeoHMSb8Osa8F3A4ikQ56dMZgsSrdbTWYvl9ofqR3/AH7H
X9g13pF5Fy7eccwFNGvvryZFJD18168XEzHNj2pdKvgqBrYYHKRlxs1bReduBKRU
O/c/dop0aWRXxreTgP4sBIawn+WbA1s3CuW900TRrvt07WKRFUdzV0zmLkUdmEHZ
mieq1QX2laBmfp729jVlNoe2v4NbdwClbJV0LGuE+4qaaCYX7VHJrQQ2RgRALpax
9SguixaC/rtbvsKSLzD7Itgm45eZqaxqf+eDpaD4v/NgENu0oH1SHJtthpyuOQTj
/D3ajuasg0P44jyytAmQNjcFIfe1sI+GwXZc2plztfewr9zuSU1rDRBWVomDrURz
c6+2frZi2FUBKDEccewE6gM7lHDY9I6tQ+yo7bXveD3OqQPtIFsShMbNwRMcowgC
D2UIH4o1msRyroYnw7OXeEyKuFkXo/7OC9g4GFXcGFKcWNR92UKqxYH60J9tRDxx
D32bCvcjhKIRMDnagA90xxNKFR5ASxYl/xvl8Bsf2mnbTI3hDt6SBaNni3qsNUAS
C1ufhkxx8nZU9qAci0JO
=htQ2
-----END PGP SIGNATURE-----

Revision history for this message
Alexander Belchenko (bialix) wrote :

Yes, we don't have get_user method in QBzr. This is filed as bug https://bugs.launchpad.net/qbzr/+bug/421851

But, Jelmer, you clearly said that on Windows bzr-svn don't use this interface. Maybe you have really serious reason to do this, but it clearly break GUI tools, especially TortoiseBzr, because it works without terminal.

Changed in qbzr:
importance: Undecided → High
status: Invalid → Confirmed
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

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

Alexander Belchenko wrote:
> Yes, we don't have get_user method in QBzr. This is filed as bug
> https://bugs.launchpad.net/qbzr/+bug/421851
>
> But, Jelmer, you clearly said that on Windows bzr-svn don't use
> this interface. Maybe you have really serious reason to do this,
> but it clearly break GUI tools, especially TortoiseBzr, because it
> works without terminal.
No, I said that on Windows we *also* use another function call that is
provided by the Subversion libraries. This function call does not do
any prompting whatsoever, but retrieves passwords from a
(windows-specific) directory that is used for caching passwords by
Subversion.

Cheers,

Jelmer
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10rc1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQIcBAEBAgAGBQJKnmsEAAoJEACAbyvXKaRXAtgP/Rcx8/u4PkZbjj3G0g+jzyzl
ILd3ZgC+SWwdi2WvF0Ez+8zVC6u1pTsR6MQ1mdf/uaplHdIXHeFvU0Jal/Y0Teaj
EMvDyGbGemIaKPcena5D//j1WhAhSm4xkNfxqhp7lizogN2IbSxBULZiH7rWpd6o
gcXIMSzPZF1rJ2047qZTEo2rnWbU5x5gine7EtE6ZHelZVD7Luiu7dYZWvaxapbL
9e1/tpZktq5enjhlB2cN59ob6kZVJLs14uIES3pcBzIls05b1V7DeOFcjXHYz3R0
0J9WJsuexBW1PGmVOf2NZnAY2PlQ/tRpQ1lOjvpsJf/+7pD44Dse7AY3rKfBU6AK
elCFeKqExr6ksh7arkWqbPdIDtRH0zTw0RBS5CWwFVKwj9nfx4FUFM89anDjFShB
9RFpYg7iWe6I2YzOjymhf7uW5DMxSLGrQdoryoqtb0AnS4l50oGKkAHqCIG2MMRX
rqehdusqsxlOqvSHjiuCiL1HJMOduw2OpCleKQxMQHDPMPg7aO6y29Ji3lLRAHTx
S0j3LsrQobLf8x8As/aAwGsMoypqUGrmGVEiV++iOIhM6BjugQBtg10jLgWsNCuX
cljHfRPTyRy3HLp/gzFP7iILq+71zfBdq/5hZHpDJQe/4BFMm0nb51QRY1mzdOsb
BaJXUZjkCh37peKIQAoH
=omSB
-----END PGP SIGNATURE-----

Revision history for this message
Alexander Belchenko (bialix) wrote :

Jelmer Vernooij пишет:
> Alexander Belchenko wrote:
>> Yes, we don't have get_user method in QBzr. This is filed as bug
>> https://bugs.launchpad.net/qbzr/+bug/421851
>>
>> But, Jelmer, you clearly said that on Windows bzr-svn don't use
>> this interface. Maybe you have really serious reason to do this,
>> but it clearly break GUI tools, especially TortoiseBzr, because it
>> works without terminal.
> No, I said that on Windows we *also* use another function call that is
> provided by the Subversion libraries. This function call does not do
> any prompting whatsoever, but retrieves passwords from a
> (windows-specific) directory that is used for caching passwords by
> Subversion.

But some code should write the password to the cache first, right?

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

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

Alexander Belchenko wrote:
> Jelmer Vernooij пишет:
>> Alexander Belchenko wrote:
>>> Yes, we don't have get_user method in QBzr. This is filed as
>>> bug https://bugs.launchpad.net/qbzr/+bug/421851
>>>
>>> But, Jelmer, you clearly said that on Windows bzr-svn don't use
>>> this interface. Maybe you have really serious reason to do
>>> this, but it clearly break GUI tools, especially TortoiseBzr,
>>> because it works without terminal.
>> No, I said that on Windows we *also* use another function call
>> that is provided by the Subversion libraries. This function call
>> does not do any prompting whatsoever, but retrieves passwords
>> from a (windows-specific) directory that is used for caching
>> passwords by Subversion.
>
> But some code should write the password to the cache first, right?
Yes, but that code isn't in bzr-svn. Only Subversion writes to that
cache since AuthenticationConfig doesn't have any infrastructure for
storing known valid passwords yet.

Cheers,

Jelmer
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10rc1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQIcBAEBAgAGBQJKnnAOAAoJEACAbyvXKaRXGzoQAJao7bnjkrmRZjv3R3BnkHEx
C98pDCnFWPCdHPGALAKTOy0pheofL+fSQ0VcassToPsaZQQDF2LB8EeHIwF/5nU7
a2zxD37yKjGnBSk4bCMwQuxAVAB3ZbZrNY99Bd1G3tTmiCGX8iY17gRQ6wIhkozc
Glep34ZENa2CVpNOppAM9AjiAZikei9psZNVD6kLqhzwHuDgpobAbECm+/a7kQFv
gUpAKZcrbkutImJcpy5uPi5rlsn2k7XrIeSzgL/0a5N6f+1QkvAssxlnwawkAMkM
MyVy1FejxM0pZOz1desrXL9NuUNa4COIQ3Mryx9U7tu/xQ2MqGhRK2TQowh3mbjS
Z10FQdCZ5jOY5toB76UH3eREWXJSvyFTzxqvMOIKwNG2hzDqF3J/5RrKQWLhzWYE
wlyoOUN1TMsTTfJ6etMwHw8EbQ9LQmJn0JDYNZcqUpvyX46rMJcMdasUpgI4xsf+
Ao48Dm3VU54ZNUhzSpqEbix80TfJAhppnx9PiVU9uOQL2qtPpvmzR6p26mvUlifN
BOFIDsFffw4zZ1Xp6jQxA8a7Rpdj6Zqaz/OnU1/bNKbyM1CpGaCLAGm2T4FBnFiA
jwbG0Gon90qAEspt5fFFrWIsLGkXb3OrdgaBDOuPB8ZUOLvJU8GYbEkx72mexLd6
fzxcwt4O3ZHLh/EspV4P
=PrMw
-----END PGP SIGNATURE-----

Revision history for this message
Alexander Belchenko (bialix) wrote :

For testing future fix for get_user support in QBzr I need to find the way how to reproduce this bug. Is there any public open svn repo which force user name prompt for testing?

Revision history for this message
Gary van der Merwe (garyvdm) wrote :
methane (songofacandy)
Changed in tortoisebzr:
status: New → Incomplete
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

This happens if you are trying to access the same Subversion repository from multiple instances of Bazaar, and are using the sqlite cache backend rather the tdb one. The TDB backend is automatically used if TDB (http://tdb.samba.org) is installed and allows concurrent access to the cache, whereas the SQLite one does not.

Revision history for this message
Gary van der Merwe (garyvdm) wrote :

I believe this is fixed by bug 421851. Please reopen if not.

Changed in qbzr:
assignee: nobody → Gary van der Merwe (garyvdm)
milestone: none → 0.16
status: Confirmed → Fix Released
Changed in bzr-svn:
status: Incomplete → Invalid
Changed in tortoisebzr:
status: Incomplete → Invalid
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.