Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[cosmos-dev] RE: BIRT references in org.eclipse.cosmos.releng.builder

Hi Brad,

 

That’s a bug.  BIRT was changed in the 1.0 release cycle to be a pre-requisite so COSMOS adopters need to download it prior to using it in the COSMOS demo.  It looks like the change to remove BIRT was made on 6/16/08 but the corresponding unzip task was not commented out or removed.

 

I’m guessing that the unzip target is throwing an error given the zip isn’t downloaded and that we are ignoring that error as part of the build.

 

I believe the fix here is to either remove the BIRT references completely (my preference to avoid confusion later) or to comment out the unzip task.  It looks like the tasks for some of the other pre-requisites, like Axis2, were properly commented out so this one was just missed.

 

Thanks,

-Jason

 

From: cosmos-dev-bounces@xxxxxxxxxxx [mailto:cosmos-dev-bounces@xxxxxxxxxxx] On Behalf Of Beck, Bradley C
Sent: Monday, April 27, 2009 11:29 AM
To: Cosmos Dev
Subject: [cosmos-dev] BIRT references in org.eclipse.cosmos.releng.builder

 

I’m confused by the references to the BIRT runtime in the builder project (org.eclipse.cosmos.releng.builder).

 

The following references are made in runtime/build.properties:

# BIRT runtime

birtURL=http://download.eclipse.org/birt/downloads/drops/R-R1-2_3_0-200806191030

birtFilename=birt-runtime-2_3_0.zip

 

While runtime/customTargets.xml contains the following references:

            <!--  <antcall target="fetch.unzip">

                  <param name="url" value="${birtURL}"/>

                  <param name="filename" value="${birtFilename}"/>

                  <param name="dstdir" value="${baseLocation}/.."/>

            </antcall> -->

            <unzip dest="${baseLocation}" overwrite="true" src="" />

 

The raises a few questions:

1)      Is the BIRT runtime still required for the COSMOS build?

2)      If BIRT is required, why has the “fetch.unzip” task for it been commented out?

3)      Why is the file “birt-runtime-2.3RC4.zip” being referenced in the “unzip” task instead of the file defined by “birtFilename” in build.properties?

 

If this is a bug, let me know and I’ll open a bugzilla for it.

 

Thanks,

-Brad


Back to the top