Clojure

Building Clojure (git head) on JDK7 fails

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    linux, "1.7.0_02-ea", x86-64

Description

lnostdal@blackbox:~/programming/clojure$ ant
Buildfile: /home/lnostdal/programming/clojure/build.xml

clean:
[delete] Deleting directory /home/lnostdal/programming/clojure/target
[delete] Deleting /home/lnostdal/programming/clojure/clojure-1.4.0-master-SNAPSHOT.jar
[delete] Deleting /home/lnostdal/programming/clojure/clojure.jar

init:
[mkdir] Created dir: /home/lnostdal/programming/clojure/target/classes
[mkdir] Created dir: /home/lnostdal/programming/clojure/target/classes/clojure

compile-java:
[javac] /home/lnostdal/programming/clojure/build.xml:39: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
[javac] Compiling 144 source files to /home/lnostdal/programming/clojure/target/classes
[javac] javac: target release 1.5 conflicts with default source release 1.7

BUILD FAILED
/home/lnostdal/programming/clojure/build.xml:39: Compile failed; see the compiler error output for details.

Total time: 0 seconds

lnostdal@blackbox:~/programming/clojure$ emacs -nw build.xml
lnostdal@blackbox:~/programming/clojure$ git diff
diff --git a/build.xml b/build.xml
index a693e6d..c05adf8 100644
— a/build.xml
+++ b/build.xml
@@ -36,7 +36,7 @@
<target name="compile-java" depends="init"
description="Compile Java sources.">
<javac srcdir="${jsrc}" destdir="${build}" includeJavaRuntime="yes"

  • debug="true" target="1.5"/>
    + debug="true"/>
    </target>

<target name="compile-clojure"

lnostdal@blackbox:~/programming/clojure$ ant
...
...

..and it builds OK. I don't understand the Java tools very well, but I decided to try removing "target" based on what I found here: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6856165

Activity

Hide
Tassilo Horn added a comment -

Exactly, Stuart. Here is a patch.

Show
Tassilo Horn added a comment - Exactly, Stuart. Here is a patch.
Hide
Stuart Halloway added a comment -

Based on the Sun bug link above, it seems to me we need to add the source="1.5" flag if we want to be able to build on 1.7

Show
Stuart Halloway added a comment - Based on the Sun bug link above, it seems to me we need to add the source="1.5" flag if we want to be able to build on 1.7

People

Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: