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

Re: [PATCH v2] Remove NLS check for KWallet support and reorder private includes

From: Raphael Kubo da Costa <rakuco_at_FreeBSD.org>
Date: Thu, 11 Oct 2012 11:29:46 +0300

"Gavin Baumanis" <gavinb_at_thespidernet.com> writes:

> Ping.
> This message has received no comments.

Ping again? :-)

> Gavin.
>
> -----Original Message-----
> From: Raphael Kubo da Costa [mailto:rakuco_at_FreeBSD.org]
> Sent: Saturday, 15 September 2012 16:11
> To: dev_at_subversion.apache.org
> Subject: [PATCH v2] Remove NLS check for KWallet support and reorder private
> includes
>
> Following Daniel's suggestions in previous thread, attached is another
> iteration of the patch that removes the need for NLS to be enabled for
> KWallet support to be built.
>
> Besides removing the NLS check from kwallet.m4 as the previous patch did,
> this one also reorders the includes in kwallet.cpp so that the private
> Subversion includes come after the system/KDE ones. The same has been done
> to the GNOME Keyring code, as well as another file.
>
> In most of the cases, svn_private_config.h is already one of the last files
> to be included, and I felt it was too dangerous to change the cases where it
> was included before some feature checks or apr includes.
Received on 2012-10-11 10:40:19 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.