-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SD-92 preview #52
base: master
Are you sure you want to change the base?
SD-92 preview #52
Conversation
A couple of comments. I don't mind using the IronJacamar XSDs as a test resource, however the official generated files will be within the IronJacamar project and the org.jboss.jca namespace. So the service files, and XSDs needs to be moved to src/test instead. The official EE descriptors (1.0, 1.5 and 1.6) can stay. An Apache Ant task and a Maven Mojo for the generator would be nice. |
I think, this is almost designed in this way. In the properties file, you can specify the following:
In the context xml, you can bundle XSD's which should belong into same context, say Java EE 6 specs or all ironjacamar XSDs, whatever you like. In this xml file, you can define as well the package names. I have included the IronJacamar XSD's only for this preview, later we should move it to a test area for regression testing. Can you have a look at the pom.xml in the cli folder? There is a maven exec plugin that allows the execute the tool including passing of a configuration file. I hope this fullfills your requirements. Of course, the maven dependencies have to be configured accordingly. |
Ok. The Apache Ant task and Maven Mojo was just for easier integration in build environments. |
I'll look to dig in here at the top of the week when back from JavaOne and team meetings. :) |
Cool:-) I added in the meantime test cases for the datasource and resource adapter descriptors. All ironjacamar xsd's are now supported. I am starting with the dtd. There are two options I am evaluating:
I have the feeling that option 2 is better. |
Can we schedule an IRC meeting for Tuesday w/ me, Ralf, and Jesper to discuss this? Propose 10AM EDT/4PM GMT+2. |
…ector_1_0.dtd as converted xsd
…ector_1_0.dtd as converted xsd
That is fine for me (it is 16:00 GMT) I have implemented a version that supports connector_1_0.dtd via conversion to an xsd. Static test case is implemented as well. Additional question:
|
Is the IRC meeting confirmed? |
I'll be at the IRC meeting, yep. :) |
This is now living https://github.com/shrinkwrap/descriptors/tree/SHRINKDESC-92. Let's keep all development in our local forks synced there. And I'll also rebase this w/ upstream/master as appropriate. |
I opened a discussion in the developer forum: http://community.jboss.org/thread/173534?tstart=0 |
Hi Andrew
This is a preview on SD-92.
I have done so far the following:
a) First option via maven exec plugin. You can find in the cli/pom.xml an example that generates the
java classes.
b) Second option, with mvn assembly:assembly we can create a jar that contains all required classes.
I think this would be very handy for the users.
almost everything. Even metadata.xslt and ddJava.xslt can be externalized. Of course, you can define the
output folder for all three packages individually. I the most simplest case, only the context xml file is
required (the schemaXXX.xml). In case of the assembled jar, the required xslt are extracted from the jar,
copied to a configurable working folder, and then the generation is starting.
There are couple of questions:
the ironjacamar xsd are generated in the jboss context. But cli already generates these descriptors
by executing the cli main class as part of the compile phase. Cool, the test cases are executed nicely,
but I couldn't make the static test runnable. Comes later.
responsibility of the user.
Let me know what you think.
Cheers,
Ralf