You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4cxx-dev@logging.apache.org by ts...@apache.org on 2014/02/14 18:30:05 UTC
svn commit: r1568372 - /incubator/log4cxx/trunk/src/ant/apr-build.xml
Author: tschoening
Date: Fri Feb 14 17:30:05 2014
New Revision: 1568372
URL: http://svn.apache.org/r1568372
Log:
Workaround for apr_atomic.c and VS2013 needed.
Modified:
incubator/log4cxx/trunk/src/ant/apr-build.xml
Modified: incubator/log4cxx/trunk/src/ant/apr-build.xml
URL: http://svn.apache.org/viewvc/incubator/log4cxx/trunk/src/ant/apr-build.xml?rev=1568372&r1=1568371&r2=1568372&view=diff
==============================================================================
--- incubator/log4cxx/trunk/src/ant/apr-build.xml (original)
+++ incubator/log4cxx/trunk/src/ant/apr-build.xml Fri Feb 14 17:30:05 2014
@@ -63,6 +63,8 @@
<property name="lib.prefix"
value=""
/>
+<property environment="env"
+/>
<target name="usage"
description="Displays usage notes on build">
@@ -236,6 +238,17 @@
value="setmode"
/>
</defineset>
+ <!--
+ apr_atomic.c is not linkable in VS2013 x86 and the only somewhat easy workaround was
+ to claim we are not VS2013, which only works because what we use to claim this is not
+ used elsewhere and doesn't seem to introduce any other problems for now.
+
+ https://groups.google.com/forum/#!topic/aprdev/bc1ILvUe-fk
+ https://groups.google.com/forum/#!topic/unimrcp/Iybpn51UYnI
+ -->
+ <defineset define="__MINGW32__"
+ if="env.VS120COMNTOOLS"
+ />
<compilerarg value="${pic-option}"
if="pic-option"