Author: tschoening
Date: Sun Sep  7 15:01:19 2014
New Revision: 1623040

URL: http://svn.apache.org/r1623040
Log:
LOGCXX-421: We have a user who wants to build log4cxx under Windows XP, but the 
current work to get C++ Builder XE 4 supported introduced a dependency on 
where.exe, which is not available under Windows XP out of the box. So I had two 
options: Either remove the added logic or make the use of where.exe optional. I 
decided the first, because the already implemented logic already seems hard to 
maintain and simply doesn't work, it looks like some changes to cpptasks is 
necessary anyways. If that's the case, I could fix all problems directly in 
cpptasks, which surely is easier than using ANT. And as always, no work done is 
lost, we can always revert...

Removed:
    incubator/log4cxx/trunk/src/ant/ilink32-workaround.xml
Modified:
    incubator/log4cxx/trunk/build.xml
    incubator/log4cxx/trunk/src/ant/apr-build.xml
    incubator/log4cxx/trunk/src/ant/common.xml

Modified: incubator/log4cxx/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/incubator/log4cxx/trunk/build.xml?rev=1623040&r1=1623039&r2=1623040&view=diff
==============================================================================
--- incubator/log4cxx/trunk/build.xml (original)
+++ incubator/log4cxx/trunk/build.xml Sun Sep  7 15:01:19 2014
@@ -81,7 +81,6 @@
 <!ENTITY find-apr SYSTEM 'src/ant/find-apr.xml'>
 <!ENTITY find-apr-util SYSTEM 'src/ant/find-apr-util.xml'>
 <!ENTITY find-libesmtp SYSTEM 'src/ant/find-libesmtp.xml'>
-<!ENTITY ilink32Workaround SYSTEM 'src/ant/ilink32-workaround.xml'>
 ]>
 
 <!--
@@ -94,7 +93,6 @@
        &find-apr;
        &find-apr-util;
        &find-libesmtp;
-       &ilink32Workaround;
 
        <property       name="base.dir"
                                location="."
@@ -971,11 +969,6 @@
 
                <antcall target="update-rc-files-version-info" />
 
-               <!--
-                       Linking doesn't work for bcc and is implemented 
manually afterwards, but because
-                       compiling works we don't want to unnecessary double 
code and simply ignore linkage
-                       errors until there's a way to explicitly don't link if 
bcc.
-                -->
                <cc     name="${project.compiler}"
                        exceptions="true"
                        
outfile="${log4cxx.lib.dir}/${lib.prefix}${lib.name}${lib.suffix}"
@@ -1049,7 +1042,6 @@
                                <dependency name="${projects.dir}/libesmtp" />
                        </project>
                </cc>
-               <antcall target="build-lib-ilink32-workaround" />
 
                <antcall target="restore-rc-files-version-info" />
 
@@ -1296,7 +1288,6 @@
                                <dependency file="${projects.dir}/log4cxx" 
depends="apr,aprutil,libesmtp" />
                        </project>
                </cc>
-               <antcall target="build-unittest-ilink32-workaround" />
        </target>
 
        <target name="build-standalone-unittest"

Modified: incubator/log4cxx/trunk/src/ant/apr-build.xml
URL: 
http://svn.apache.org/viewvc/incubator/log4cxx/trunk/src/ant/apr-build.xml?rev=1623040&r1=1623039&r2=1623040&view=diff
==============================================================================
--- incubator/log4cxx/trunk/src/ant/apr-build.xml (original)
+++ incubator/log4cxx/trunk/src/ant/apr-build.xml Sun Sep  7 15:01:19 2014
@@ -18,7 +18,6 @@
 -->
 <!DOCTYPE project [
 <!ENTITY common SYSTEM 'common.xml'>
-<!ENTITY ilink32Workaround SYSTEM 'ilink32-workaround.xml'>
 <!ENTITY license '
  Licensed to the Apache Software Foundation (ASF) under one or more
  contributor license agreements.  See the NOTICE file distributed with
@@ -43,7 +42,6 @@
 <project name="apr" default="check">
 
 &common;
-&ilink32Workaround;
 
 <property      name="src.dir"
                        location="${basedir}"
@@ -270,7 +268,6 @@
                        <comment>&license;</comment>
                </project>
        </cc>
-       <!-- antcall target="apr-build-ilink32-workaround" /-->
 
        <property       name="apr.lib.dir"
                                location="${executable.dir}"

Modified: incubator/log4cxx/trunk/src/ant/common.xml
URL: 
http://svn.apache.org/viewvc/incubator/log4cxx/trunk/src/ant/common.xml?rev=1623040&r1=1623039&r2=1623040&view=diff
==============================================================================
--- incubator/log4cxx/trunk/src/ant/common.xml (original)
+++ incubator/log4cxx/trunk/src/ant/common.xml Sun Sep  7 15:01:19 2014
@@ -94,20 +94,7 @@
                        <equals arg1="${compiler}" arg2="gcc" />
                        <equals arg1="${compiler}" arg2="g++" />
                </or>
-       </condition>
-
-       <!-- LOG4CXX-421 -->
-       <exec   executable="where"
-                       outputproperty="ilink32.path"
-                       failonerror="false">
-               <arg value="ilink32" />
-       </exec>
-       <propertyregex  property="bcc.rootPath"
-                                       input="${ilink32.path}"
-                                       regexp="^(.+?)\\bin\\ilink32.exe"
-                                       select="\1"
-                                       casesensitive="false"
-       />
+       </condition>
 </target>
 
 <target        name="mac-init"


Reply via email to