[ 
http://jira.codehaus.org/browse/MOJO-809?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Julien HENRY updated MOJO-809:
------------------------------

    Attachment: compiler-toolchains.patch

I have implemented a patch. I have duplicated most useful parameters from 
maven-compiler-plugin and I am now using plexus-compiler to perform javac. New 
parameters on the mojo are "encoding", "source", "target", "debug" and also it 
is now toolchains aware.

I did some version update in pom (mojo parent and maven api) but it was not 
required so you may revert to previous values if you like.

Note that MavenJavacCompiler is not thread safe as I am using lot of static 
variables as jasper wants to create the class dynamically. But I don't think 
this is a major issue as I don't see a use case where jasperreports mojo will 
be executed in parallel. If you have a better suggestion, feel free to tell me.

> Added support for selecting target version of the generated class files to 
> jasperreports-maven-plugin
> -----------------------------------------------------------------------------------------------------
>
>                 Key: MOJO-809
>                 URL: http://jira.codehaus.org/browse/MOJO-809
>             Project: Mojo
>          Issue Type: Improvement
>          Components: jasperreports
>            Reporter: Roland Klein
>            Priority: Minor
>         Attachments: compiler-toolchains.patch, MOJO-809-patch.txt, 
> patch.dif, patch.diff, patch.diff, patch.diff
>
>
> Attached patch for rev.4230
> + JRPluginCompiler.java
> * pom.xml
> * JasperReportsMojo.java

-- 
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