Re: [PATCH v2] Remove NLS check for KWallet support and reorder private includes
"Gavin Baumanis" <gavinb_at_thespidernet.com> writes:
> This message has received no comments.
Ping again? :-)
> -----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
> 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