Return-Path: 
 <harmony-commits-return-1932-apmail-incubator-harmony-commits-archive=incubator.apache.org@incubator.apache.org>
Delivered-To: apmail-incubator-harmony-commits-archive@www.apache.org
Received: (qmail 27820 invoked from network); 2 Apr 2006 20:16:06 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199)
  by minotaur.apache.org with SMTP; 2 Apr 2006 20:16:06 -0000
Received: (qmail 26779 invoked by uid 500); 2 Apr 2006 20:16:05 -0000
Delivered-To: apmail-incubator-harmony-commits-archive@incubator.apache.org
Received: (qmail 26747 invoked by uid 500); 2 Apr 2006 20:16:05 -0000
Mailing-List: contact harmony-commits-help@incubator.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:harmony-commits-help@incubator.apache.org>
List-Unsubscribe: <mailto:harmony-commits-unsubscribe@incubator.apache.org>
List-Post: <mailto:harmony-commits@incubator.apache.org>
List-Id: <harmony-commits.incubator.apache.org>
Reply-To: harmony-dev@incubator.apache.org
Delivered-To: mailing list harmony-commits@incubator.apache.org
Received: (qmail 26736 invoked by uid 99); 2 Apr 2006 20:16:05 -0000
Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49)
    by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 02 Apr 2006 13:16:05 -0700
X-ASF-Spam-Status: No, hits=0.0 required=10.0
	tests=
X-Spam-Check-By: apache.org
Received: from [192.87.106.226] (HELO ajax.apache.org) (192.87.106.226)
    by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 02 Apr 2006 13:16:04 -0700
Received: from ajax (localhost.localdomain [127.0.0.1])
	by ajax.apache.org (Postfix) with ESMTP id A6C676ACB1
	for <harmony-commits@incubator.apache.org>;
 Sun,  2 Apr 2006 21:15:43 +0100 (BST)
Message-ID: <1003549522.1144008943674.JavaMail.jira@ajax>
Date: Sun, 2 Apr 2006 21:15:43 +0100 (BST)
From: "Mark Hindess (JIRA)" <jira@apache.org>
To: harmony-commits@incubator.apache.org
Subject: [jira] Updated: (HARMONY-293) reduce duplication in module ant
 files
In-Reply-To: <1841350772.1144007983363.JavaMail.jira@ajax>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
X-Virus-Checked: Checked by ClamAV on apache.org
X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N

     [ http://issues.apache.org/jira/browse/HARMONY-293?page=all ]

Mark Hindess updated HARMONY-293:
---------------------------------

    Attachment: 01.ant.file.refactor.sh
                02.ant.file.refactor.diff

Tentative proposal for refactoring.  (Note: this incorporates the minor fix from HARMONY-279 since I wanted to test the tests.)

The patch creates a common module.xml build file that all the modules should import.  Specific module build.xml files can then override only the minimal amount of the common build file thus making the differences much more transparent.  Taking the archive and luni example again, the archive build.xml file doesn't override anything and becomes:

  <project name="Build" default="build" basedir="..">
    <description>Build for archive component</description>
    <import file="${basedir}/../../make/module.xml" />
  </project>

while the luni build.xml file becomes:

  <project name="Build" default="build" basedir="..">
    <description>Build for luni component</description>
    <import file="${basedir}/../../make/module.xml" />
    <target name="run.tests">
      <run-tests>
        <!-- Required by various tests that set security manager etc -->
        <jvmarg value="-Djava.security.policy=../../../../support/src/test/resources/config/testing.policy" />
        <!-- Required for running the java.net unit tests -->
        <jvmarg value="-Dtest.ini.file=../../../../src/test/resources/config/localhosttest.ini" />
      </run-tests>
    </target>
  </project>

This makes it quite clear that the difference is only that two additional arguments are passed to the jvm used to run the tests.


> reduce duplication in module ant files
> --------------------------------------
>
>          Key: HARMONY-293
>          URL: http://issues.apache.org/jira/browse/HARMONY-293
>      Project: Harmony
>         Type: Improvement

>   Components: Classlib
>     Reporter: Mark Hindess
>     Priority: Minor
>  Attachments: 01.ant.file.refactor.sh, 02.ant.file.refactor.diff
>
> At the moment it is very difficult to see the differences between the way different modules are built.
> For instance, one might try to assess the differences between the build process of archive and luni by reading the recursive diff of modules/archive/make and modules/luni/make.  The diff contains many trivial differences (e.g. search and replace archive with luni) that make it difficult to see what the really important differences are.  The common structure should be factored out to a common build file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira