These look fine for now.

I want to avoid a dependency on the SDK’s swfutils because it has a dependency 
on Batik that we’ve hacked with some non-standard modifications.

As I continue to try to get FDB to work without the SDK’s swfutils, I am 
basically copying more and more files from swfutils and hoping not to copy the 
ones that depend on Batik.  Then we’ll see we can remove the dependency on 
swfutils for falcon’s flex-oem-compiler and see if IJ still works.

So, once I check in FDB, I’ll start fiddling with our subset of swfutils and 
hopefully eventually produce a new nightly build package with a Falcon-based 
FDB and your changes to flex-oem-compiler that uses our subset of swfutils and 
then we’ll do more testing with IJ.

How does that sound?

BTW, have you tried IJ with a FlexJS SDK with your changes to flex-oem-compiler 
but without any Flex SDK mixed in?  Does it work any better than when you first 
tried it?

Thanks,
-Alex

From: Frédéric THOMAS <ftho...@apache.org<mailto:ftho...@apache.org>>
Reply-To: "dev@flex.apache.org<mailto:dev@flex.apache.org>" 
<dev@flex.apache.org<mailto:dev@flex.apache.org>>
Date: Wednesday, May 27, 2015 at 6:49 AM
To: "comm...@flex.apache.org<mailto:comm...@flex.apache.org>" 
<comm...@flex.apache.org<mailto:comm...@flex.apache.org>>
Subject: RE: [2/2] git commit: [flex-falcon] [refs/heads/IDEA-FLEX_JS_COMPILER] 
- FLEX-34860 : Updated the Ant files

Hi,

I would like the build to be reviewed, someone ?

Thanks,
Frédéric THOMAS

> From: ftho...@apache.org<mailto:ftho...@apache.org>
> To: comm...@flex.apache.org<mailto:comm...@flex.apache.org>
> Date: Wed, 27 May 2015 13:45:30 +0000
> Subject: [2/2] git commit: [flex-falcon] [refs/heads/IDEA-FLEX_JS_COMPILER] - 
> FLEX-34860 : Updated the Ant files
>
> FLEX-34860 : Updated the Ant files
>
> - NOTE: I needed swfutils.jar from the FLEX_HOME, the current kind of copy 
> sources didn't work for me, this need eventually to be reviewed.
>
>
> Project: http://git-wip-us.apache.org/repos/asf/flex-falcon/repo
> Commit: http://git-wip-us.apache.org/repos/asf/flex-falcon/commit/f47c8490
> Tree: http://git-wip-us.apache.org/repos/asf/flex-falcon/tree/f47c8490
> Diff: http://git-wip-us.apache.org/repos/asf/flex-falcon/diff/f47c8490
>
> Branch: refs/heads/IDEA-FLEX_JS_COMPILER
> Commit: f47c8490cd2018325dd4fae9766ebcd690d61c7f
> Parents: a6f360e
> Author: Frédéric THOMAS <webdoubl...@gmail.com<mailto:webdoubl...@gmail.com>>
> Authored: Wed May 27 14:44:56 2015 +0100
> Committer: Frédéric THOMAS 
> <webdoubl...@gmail.com<mailto:webdoubl...@gmail.com>>
> Committed: Wed May 27 14:44:56 2015 +0100
>
> ----------------------------------------------------------------------
> build.xml | 6 +++---
> flex-compiler-oem/build.xml | 35 ++++++++++++++++++++++++++++++++---
> 2 files changed, 35 insertions(+), 6 deletions(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/flex-falcon/blob/f47c8490/build.xml
> ----------------------------------------------------------------------
> diff --git a/build.xml b/build.xml
> index 3a8b300..7447397 100644
> --- a/build.xml
> +++ b/build.xml
> @@ -66,7 +66,7 @@
> <ant dir="compiler" target="sdk"/>
> </target>
>
> - <target name="main" depends="sdk, javadoc, tests" description="Default 
> target - Builds a Falcon SDK, builds Falcon Javadoc, and runs tests."/>
> + <target name="main" depends="sdk, javadoc" description="Default target - 
> Builds a Falcon SDK, builds Falcon Javadoc, and runs tests."/>
>
> <target name="all" depends="main, jx, oem" description="Builds Falcon, then 
> FalconJX"/>
>
> @@ -82,7 +82,7 @@
> <ant dir="compiler.tests" target="main"/>
> </target>
>
> - <target name="compiler.oem" depends="swfutils" description="Builds FB 
> Integration JAR">
> + <target name="compiler.oem" depends="compiler.jx, swfutils" 
> description="Builds FB Integration JAR">
> <ant dir="flex-compiler-oem" target="main"/>
> </target>
>
> @@ -136,7 +136,7 @@
> <available file="swfutils"
> type="dir"
> property="swfutils.bundled"
> - value="true" />
> + value="false" />
> </target>
>
> <target name="compiler.jx" description="Builds FalconJX.">
>
> http://git-wip-us.apache.org/repos/asf/flex-falcon/blob/f47c8490/flex-compiler-oem/build.xml
> ----------------------------------------------------------------------
> diff --git a/flex-compiler-oem/build.xml b/flex-compiler-oem/build.xml
> index cded834..3833416 100644
> --- a/flex-compiler-oem/build.xml
> +++ b/flex-compiler-oem/build.xml
> @@ -79,6 +79,8 @@
>
> <path id="classpath">
> <fileset dir="${sdk}/lib" includes="**/*.jar"/>
> + <fileset dir="U:/sources/asf/flex/falcon/compiler.jx/lib" 
> includes="jsc.jar"/>
> + <fileset dir="${FLEX_HOME}/lib" includes="swfutils.jar"/>
> </path>
>
>
> @@ -98,7 +100,6 @@
> source="${javac.src}" target="${javac.src}"
> includes="**/*.java" destdir="${oem}/bin" classpathref="classpath" 
> includeAntRuntime="true">
> <src path="${oem}/src"/>
> - <src path="${basedir}/../swfutils/src"/>
> </javac>
> <!--
> <copy todir="${compiler}/generated/classes">
> @@ -106,6 +107,33 @@
> </copy>
> -->
> </target>
> +
> +
> + <target name="set.compc.jar.uptodate">
> + <uptodate property="compc.jar.uptodate"
> + targetfile="${sdk}/lib/compc.jar">
> + <srcfiles dir="${compiler}/generated/classes">
> + <include name="**/*.class"/>
> + <include name="**/*.properties"/>
> + </srcfiles>
> + </uptodate>
> + </target>
> +
> + <target name="compc.jar" depends="oem.jar,set.compc.jar.uptodate" 
> unless="compc.jar.uptodate">
> + <mkdir dir="${sdk}/lib"/>
> + <jar file="${sdk}/lib/compc.jar" basedir="${compiler}/generated/classes" 
> whenmanifestonly="create">
> + <include name="META-INF/LICENSE"/>
> + <include name="META-INF/NOTICE"/>
> + <manifest>
> + <attribute name="Sealed" value="${manifest.sealed}"/>
> + <attribute name="Implementation-Title" 
> value="${manifest.Implementation-Title} - COMPC Command Line Compiler"/>
> + <attribute name="Implementation-Version" 
> value="${manifest.Implementation-Version}.${build.number}"/>
> + <attribute name="Implementation-Vendor" 
> value="${manifest.Implementation-Vendor}"/>
> + <attribute name="Main-Class" value="flex2.tools.Compc"/>
> + <attribute name="Class-Path" value="flex-compiler-oem.jar"/>
> + </manifest>
> + </jar>
> + </target>
>
> <target name="set.oem.jar.uptodate">
> <uptodate property="oem.jar.uptodate"
> @@ -126,12 +154,13 @@
> <attribute name="Implementation-Title" 
> value="${manifest.Implementation-Title}"/>
> <attribute name="Implementation-Version" 
> value="${manifest.Implementation-Version}.${build.number}"/>
> <attribute name="Implementation-Vendor" 
> value="${manifest.Implementation-Vendor}"/>
> - <attribute name="Class-Path" value="falcon-mxmlc.jar falcon-compc.jar 
> swfutils.jar"/>
> + <attribute name="Main-Class" value="flex2.tools.Mxmlc"/>
> + <attribute name="Class-Path" value="compiler.jar swfutils.jar 
> ../js/lib/jsc.jar"/>
> </manifest>
> </jar>
> </target>
>
> - <target name="jar" depends="oem.jar"
> + <target name="jar" depends="oem.jar, compc.jar"
> description="Creates JAR files"/>
>
> <target name="sdk" depends="jar" description="Builds a Falcon SDK"/>
>

Reply via email to