[Maven-plugins-developer] [ maven-plugins-Bugs-1336145 ] [jaxb] does not use dependency classpath

[SourceForge.net <no...@so...>]

Bugs item #1336145, was opened at 2005-10-24 03:54
Message generated for change (Comment added) made by jeffjensen
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=497853&aid=1336145&group_id=61626

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: jaxb
Group: release
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Jeff Jensen (jeffjensen)
Summary: [jaxb] does not use dependency classpath

Initial Comment:
When using the JAXB plugin with MAVEN 1.0.2, the
project dependencies classpath is ignored. This is
because the plugin.jelly is incorrect.

It says :      
<xjc extension="${maven.jaxb.extension}"
package="${maven.jaxb.package}"
          target="${maven.jaxb.build.dir}">
        <schema dir="${maven.jaxb.src.dir}"
            includes="${maven.jaxb.schema.includes}"
            excludes="${maven.jaxb.schema.excludes}"/>
        <binding dir="${maven.jaxb.src.dir}"
            includes="${maven.jaxb.bindings.includes}"
            excludes="${maven.jaxb.bindings.excludes}"/>
        <ant:classpath>
          <ant:pathelement path="${maven.build.dest}" />
          <ant:pathelement
path="${maven.dependency.classpath}" />
        </ant:classpath>
      </xjc>

While in fact, the classpath should use a refid like so :

          <ant:path refid="maven.dependency.classpath" />



----------------------------------------------------------------------

>Comment By: Jeff Jensen (jeffjensen)
Date: 2006-07-30 17:23

Message:
Logged In: YES 
user_id=498804

Sorry - just found your patch on the patches list.  Thanks a
bunch for doing that.  It is applied and I will release a
snapshot for it.

----------------------------------------------------------------------

Comment By: Jeff Jensen (jeffjensen)
Date: 2006-07-30 16:12

Message:
Logged In: YES 
user_id=498804

Are you saying to replace this:
        <ant:classpath>
          <ant:pathelement path="${maven.build.dest}" />
          <ant:pathelement
path="${maven.dependency.classpath}" />
        </ant:classpath>

with this:
  <ant:path refid="maven.dependency.classpath" />

?  If not, please clarify precisely the change you are
recommending, or even better please attach a patch!



----------------------------------------------------------------------

Comment By: Brice C. (brice_c)
Date: 2005-10-24 03:55

Message:
Logged In: YES 
user_id=65157

submitted by SF user brice_c

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=497853&aid=1336145&group_id=61626