← Back to team overview

svn team mailing list archive

Re: Preparing to merge with upcoming Subversion 1.6.12dfsg-1 from Debian testing

 

On 2010-07-01 14:22, Michael Diers wrote:
> On 2010-07-01 08:48, Max Bowsher wrote:
>> On 30/06/10 15:53, Michael Diers wrote:
>>> However, there are no extra runtime dependencies involved when support
>>> for GNOME Keyring and KWallet is enabled. The dependency is strictly
>>> build-time. Subversion will probe for the libraries at runtime; if they
>>> are not installed -- well, that's fine, too. Subversion will simply
>>> revert to storing secrets in the user's home directory.
>>
>> Aha. I didn't realize that there was neither run-time nor
>> package-install-time dependency involved. That makes things much better.
>>
>>> So unless we want to deliberately protect users of Karmic and below from
>>> the change of behaviour that the KWallet integration entails, when the
>>> required libraries are installed, I don't see the point of explicitly
>>> disabling it. In fact, I'm very much in favour of enabling it.
>>
>> It's an issue, but a minor one. I now agree with you that we should turn
>> it back on (assuming, of course, KDE in all the older distroseries is
>> new enough for Subversion's needs?)
> 
> I understand that this is in fact the case, i.e. the KWallet API has
> been stable since Hardy at least.

Oh, rats. I'll give up on bringing KWallet integration to Hardy in our PPA.

While the KWallet API has been stable for a long time, Subversion 1.6
only supports detecting KWallet via a KDE 4.x feature:

  ...
  checking for kde4-config... no
  configure: error: cannot find kde4-config
  ...

Hardy has KDE 3.x in Main; with KDE 4.x only available in Universe.
However, packages from Universe rightly are not available for our
Launchpad PPA builds...

Max, please do go ahead with the 1.6.12 merge. I'll provide a custom
KWallet-enabled build for my remaining Hardy users.

-- 
Michael Diers, elego Software Solutions GmbH, http://www.elego.de



Follow ups

References