Menu

#61 [jaxb] does not use dependency classpath

release
closed-fixed
jaxb (1)
5
2006-07-30
2005-10-24
Anonymous
No

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

Discussion

  • Brice C.

    Brice C. - 2005-10-24

    Logged In: YES
    user_id=65157

    submitted by SF user brice_c

     
  • Jeff Jensen

    Jeff Jensen - 2006-07-30
    • assigned_to: nobody --> jeffjensen
    • summary: JAXB plugin does not use dependency classpath --> [jaxb] does not use dependency classpath
    • status: open --> pending
     
  • Jeff Jensen

    Jeff Jensen - 2006-07-30

    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!

     
  • Jeff Jensen

    Jeff Jensen - 2006-07-30

    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.

     
  • Jeff Jensen

    Jeff Jensen - 2006-07-30
    • status: pending --> closed-fixed
     

Log in to post a comment.