[vincent_aumont]

Hi Ales,

How difficult do you think it'd be to fix the deployer?

[alesj]

How difficult do you think it'd be to fix the deployer?

Shouldn't be too difficult.

Most of the logic is already there, you would just need to properly handle multiple matching files -- this logic is missing atm.

Then you also need to check what next JPA handling deployers expect; e.g. what kind of attachment, under which key, etc.

That might mean you'll have to change more then just parsing deployer,

but the changes should still be simple.

You're more than welcome to try. ;-)

[vincent_aumont]

I can give it a shot. What class should I start with?

[alesj]

I can give it a shot. What class should I start with?

This are all JPA deployers:

* http://anonsvn.jboss.org/repos/jbossas/trunk/ejb3/src/resources/META-INF/jpa-deployers-jboss-beans.xml

This is the one we need to change first:

* org.jboss.jpa.deployers.PersistenceParsingDeployer

The actual code is here:

* http://anonsvn.jboss.org/repos/jbossas/projects/jpa/trunk/

[vincent_aumont]

Hmmm, should niot it be SchemaResolverDeployer (or one of its super-class)  that I need to change first?

I tried to build  http://anonsvn.jboss.org/repos/jbossas/trunk but got this exception:

{code}
Failed to execute goal org.codehaus.mojo:license-maven-plugin:1.0-alpha-1-2-jbossorg:download-licenses 
(default) on project jboss-as-build: Unable to parse license summary output file. NullPointerException
{code}

[alesj]

Hmmm, should niot it be SchemaResolverDeployer (or one of its super-class)  that I need to change first?

No, it's all already there, we're just missing a few details. ;-)

* single name + no-suffix will result in a single found file --> simply adding "" suffix will match multiple files

* we need to set allowMultipleFiles == true

* each parse result will go into unit's attachment under "unit.addAttachment(file.toURL().toString(), result, getOutput());"

** we need to check if next JPA deployers understand this kind of attachment name

*** if not, we need to change the way they lookup JPA PU attachment(s)

[alesj]

I tried to build  http://anonsvn.jboss.org/repos/jbossas/trunk but got this exception:

Failed to execute goal org.codehaus.mojo:license-maven-plugin:1.0-alpha-1-2-jbossorg:download-licenses 
(default) on project jboss-as-build: Unable to parse license summary output file. NullPointerException 

No idea about this one.