[ 
https://issues.apache.org/jira/browse/QPID-3326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13059218#comment-13059218
 ] 

Robbie Gemmell commented on QPID-3326:
--------------------------------------

Hi Danushka,

Looking into what the patch does I think it would be more appropriate for you 
to perform this change within your POM file when running the Ant task, which 
does seem to be the recommended guidance from postings on the topic I found via 
Google. Applying the patch anyway to verify my suspicion about its perofmance 
impact, I found it increased the time for me to do a clean build by 45% due to 
all the unnecessary forks for the compile tasks. I dont think it makes sense to 
inflict that kind of performance degredation on developers just using Ant 
directly when a clean alternative exists for those using Maven to drive their 
build process.

If you could try some of the other build.compiler settings to see if they can 
fix your issue without necessitating the need to fork the compilation task then 
we could consider applying that to the Ant build files, but if extJavac is the 
only solution then I think the proper direction is just for downstream POM's to 
control this setting themselves.

I'll leave this issue open for now in case you'd like to test the alternative 
options. Let us know the results, and we can either apply an updated patch or 
just close this out.

Robbie

> Modify ant build scripts to work with Maven
> -------------------------------------------
>
>                 Key: QPID-3326
>                 URL: https://issues.apache.org/jira/browse/QPID-3326
>             Project: Qpid
>          Issue Type: Bug
>            Reporter: Danushka Menikkumbura
>            Assignee: Robbie Gemmell
>         Attachments: QPID-3326.patch
>
>


--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to