[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r31427 - trunk/subversion/libsvn_auth_kwallet

From: Stefan Sperling <stsp_at_elego.de>
Date: Tue, 27 May 2008 15:25:49 +0200

On Tue, May 27, 2008 at 03:15:27PM +0200, Arfrever Frehtes Taifersar Arahesis wrote:
> 2008-05-26 David Glasser <glasser_at_davidglasser.net>:
> > On Sat, May 24, 2008 at 10:07 AM, <arfrever_at_tigris.org> wrote:
> >> Author: arfrever
> >> Date: Sat May 24 10:07:45 2008
> >> New Revision: 31427
> >>
> >> Log:
> >> * subversion/libsvn_auth_kwallet/kwallet.cpp
> >> (kwallet_password_get, kwallet_password_set): Don't disconnect Subversion
> >> from KWallet because disconnection works incorrectly in KDE 4.0.4.
> >>
> >> Modified:
> >> trunk/subversion/libsvn_auth_kwallet/kwallet.cpp
> >>
> >> Modified: trunk/subversion/libsvn_auth_kwallet/kwallet.cpp
> >> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_auth_kwallet/kwallet.cpp?pathrev=31427&r1=31426&r2=31427
> >> ==============================================================================
> >> --- trunk/subversion/libsvn_auth_kwallet/kwallet.cpp Sat May 24 09:27:55 2008 (r31426)
> >> +++ trunk/subversion/libsvn_auth_kwallet/kwallet.cpp Sat May 24 10:07:45 2008 (r31427)
> >> @@ -104,8 +104,8 @@ kwallet_password_get(const char **passwo
> >> }
> >> }
> >> }
> >> - KWallet::Wallet::disconnectApplication(wallet_name,
> >> - QString::fromUtf8("Subversion"));
> >> +// KWallet::Wallet::disconnectApplication(wallet_name,
> >> +// QString::fromUtf8("Subversion"));
> >
> > Was this just a temporary tweak?
>
> Yes. I wait for https://bugs.kde.org/show_bug.cgi?id=162570.

What about this patch, then, for the time being?

Index: subversion/libsvn_auth_kwallet/kwallet.cpp
===================================================================
--- subversion/libsvn_auth_kwallet/kwallet.cpp (revision 31465)
+++ subversion/libsvn_auth_kwallet/kwallet.cpp (working copy)
@@ -167,6 +167,9 @@
             }
         }
     }
+// This function currently closes the wallet if no other application
+// is connected to the wallet. We're waiting for this to be fixed
+// upstream, see https://bugs.kde.org/show_bug.cgi?id=162570
 // KWallet::Wallet::disconnectApplication(wallet_name,
 // QString::fromUtf8("Subversion"));
   return ret;

Stefan

  • application/pgp-signature attachment: stored
Received on 2008-05-27 15:24:25 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.