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

Christian Schulte updated MJAXB-33:
-----------------------------------

    Attachment: MJAXB-33.patch

Updated patch adding support for quoted arguments without adding a new mojo 
parameter.

Also adds a workaround for the episode add-on. See 'mjaxb-33 episode with 
spaces'. The workaround stops XJC from throwing a FileNotFoundException when 
using '-episode .../META-INF/sun-jaxb.episode'. However, supporting separate 
compilation [1] is another issue.

If adding new mojo parameters is an option, I would prefer to add that 
'extraArguments' parameter keeping the current 'arguments' parameter backwards 
compatible. This patch will strip any quotes from quoted arguments 
intentionally. Current behaviour is to pass quotes to XJC.

The 'getArguments' method this patch adds also needs more testing.

[1] http://weblogs.java.net/blog/2006/09/05/separate-compilation-jaxb-ri-21


> Extra arguments containing spaces not supported.
> ------------------------------------------------
>
>                 Key: MJAXB-33
>                 URL: http://jira.codehaus.org/browse/MJAXB-33
>             Project: Maven 2.x JAXB 2.1 Plugin
>          Issue Type: Improvement
>    Affects Versions: 1.3
>         Environment: Apache Maven 2.0.11 (r909250; 2010-02-12 05:55:50+0000)
> Java version: 1.6.0_18
>            Reporter: Christian Schulte
>         Attachments: MJAXB-33.patch, MJAXB-33.patch
>
>
> The 'arguments' parameter of the 'xjc' and 'testXjc' goals cannot take path 
> arguments containing spaces.

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