Author: damjan
Date: Sun Apr  8 13:25:06 2018
New Revision: 1828640

URL: http://svn.apache.org/viewvc?rev=1828640&view=rev
Log:
Pass the Windows path to Ant on Windows, as /cygdrive/c/... doesn't work.

Skip tests when JUnit is absent correctly: the OOO_JUNIT_JAR variable
isn't blank but absent.

This should fix building on Windows.

Patch by: me


Modified:
    openoffice/trunk/main/solenv/ant/aoo-ant.xml
    openoffice/trunk/main/solenv/gbuild/Ant.mk

Modified: openoffice/trunk/main/solenv/ant/aoo-ant.xml
URL: 
http://svn.apache.org/viewvc/openoffice/trunk/main/solenv/ant/aoo-ant.xml?rev=1828640&r1=1828639&r2=1828640&view=diff
==============================================================================
--- openoffice/trunk/main/solenv/ant/aoo-ant.xml (original)
+++ openoffice/trunk/main/solenv/ant/aoo-ant.xml Sun Apr  8 13:25:06 2018
@@ -119,10 +119,10 @@
     </target>
 
     <target name="test-compile" depends="compile,idl">
-        <property property="test.skip" value="true" 
if:blank="${OOO_JUNIT_JAR}"/>
-        <echo message="No junit, skipping tests" if:blank="${OOO_JUNIT_JAR}"/>
+        <property name="test.skip" value="true" unless:set="OOO_JUNIT_JAR"/>
+        <echo message="No junit, skipping tests" unless:set="OOO_JUNIT_JAR"/>
 
-        <mkdir dir="${test.build.dir}" unless:blank="${OOO_JUNIT_JAR}"/>
+        <mkdir dir="${test.build.dir}" if:set="OOO_JUNIT_JAR"/>
         <javac srcdir="${test.src.dir}"
                destdir="${test.build.dir}"
                debug="${test.debug}"
@@ -130,12 +130,12 @@
                deprecation="${test.deprecation}"
                classpathref="internal.test.classpath"
                includeantruntime="false"
-               unless:blank="${OOO_JUNIT_JAR}"/>
+               if:set="OOO_JUNIT_JAR"/>
     </target>
 
     <!-- fork="true" is sadly necessary on Ubuntu due to multiple versions of 
junit confusing Ant,
          see https://github.com/real-logic/simple-binary-encoding/issues/96 -->
-    <target name="test" depends="test-compile" unless:set="${test.skip}">
+    <target name="test" depends="test-compile" unless="${test.skip}">
         <mkdir dir="${test.reports.dir}"/>
         <junit printsummary="yes" haltonfailure="yes" showoutput="true" 
filtertrace="false" fork="true">
             <classpath refid="internal.test.classpath"/>

Modified: openoffice/trunk/main/solenv/gbuild/Ant.mk
URL: 
http://svn.apache.org/viewvc/openoffice/trunk/main/solenv/gbuild/Ant.mk?rev=1828640&r1=1828639&r2=1828640&view=diff
==============================================================================
--- openoffice/trunk/main/solenv/gbuild/Ant.mk (original)
+++ openoffice/trunk/main/solenv/gbuild/Ant.mk Sun Apr  8 13:25:06 2018
@@ -50,8 +50,13 @@ FORCE:
 # adds jar files to DeliverLogTarget
 # adds dependency for outdir target to workdir target (pattern rule for 
delivery is in Package.mk)
 define gb_Ant_Ant
+ifeq ($(OS),WNT)
+$(call gb_Ant_get_target,$(1)) : ANTBUILDFILE := `cygpath -m $(2)`
+$(call gb_Ant_get_clean_target,$(1)) : ANTBUILDFILE := `cygpath -m $(2)`
+else
 $(call gb_Ant_get_target,$(1)) : ANTBUILDFILE := $(2)
 $(call gb_Ant_get_clean_target,$(1)) : ANTBUILDFILE := $(2)
+endif
 $(eval $(call gb_Module_register_target,$(call 
gb_Jar_get_outdir_target,$(1)),$(call gb_Ant_get_clean_target,$(1))))
 $(call gb_Deliver_add_deliverable,$(call gb_Jar_get_outdir_target,$(1)),$(call 
gb_Ant_get_target,$(1)),$(1))
 $(call gb_Jar_get_outdir_target,$(1)) : $(call gb_Ant_get_target,$(1))


Reply via email to