[ http://jira.codehaus.org/browse/MOJO-453?page=comments#action_79072 ] 
            
Anagnostopoulos Kostis commented on MOJO-453:
---------------------------------------------

I have upto-dated and simplified the patch, and i'm ready to submit it.  

Here i have 2 remarks that i need you to answer before attaching the patch here:

1. The method org.codehaus.mojo.xml.test.Resolver.resolveEntity() throws the 
unecessary exceptions:
  SAXException, IOException
Should i remove them, and include this fix into the patch?
(It would be a waste of time to open a new issue, unless these exceptions are 
there for a reason)

2. Test case it4 includes the target dir into svn.  Should i remove it ?

> [xml-maven-plugin] Perform up-to-date check, to avoid re-transforming files
> ---------------------------------------------------------------------------
>
>                 Key: MOJO-453
>                 URL: http://jira.codehaus.org/browse/MOJO-453
>             Project: Mojo
>          Issue Type: Improvement
>          Components: xml
>         Environment: version: 0.0.1
> svn revision: 2089
>            Reporter: Anagnostopoulos Kostis
>         Assigned To: Jochen Wiedmann
>         Attachments: xmlplgn.upToDateChk-MOJO-453.patch
>
>
> The plugin does not test whether the result files of the 'transform' goal are 
> up-to-date, and avoid re-processing them.  This is necessary since ussually 
> XSLT transformations take a long time to complete (i.e. in forked lifecycles, 
> the delay is important).

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

        

---------------------------------------------------------------------
To unsubscribe from this list please visit:

    http://xircles.codehaus.org/manage_email

Reply via email to