<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Stuart,</div><div><br></div>That's great, thanks a lot. I'll keep an eye on the PR. <div><br></div><div>Also, won't I still need to use the WeldDeploymentMarker?</div><div><br></div><div>Paul.</div><div><br><div><div><div>On 22 May 2013, at 01:38, Stuart Douglas <<a href="mailto:stuart.w.douglas@gmail.com">stuart.w.douglas@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">I have been meaning to change the way these are registered for some time:<br><br><a href="https://github.com/wildfly/wildfly/pull/4535">https://github.com/wildfly/wildfly/pull/4535</a><br><br>Now it should just be a matter of adding a dependency with a services import on the relevant module.<br><br>Stuart<br><br>Paul Robinson wrote:<br><blockquote type="cite">Stuart,<br><br>The problem with this approach is that I need to introduce a dependency<br>on org.jboss.as:jboss-as-weld, which depends on<br>org.jboss.as:jboss-as-transactions. This introduces a cyclic dependency.<br>org.jboss.as:jboss-as-weld is needed for the WeldDeploymentMarker and to<br>use WeldAttachments.PORTABLE_EXTENSIONS.<br><br><blockquote type="cite">You can do this for every deployment, if beans.xml is not present it<br>won't do anything<br></blockquote><br>I assume this is done by the three tests at the top of the deploy<br>method? Therefore, I would need to use WeldDeploymentMarker.<br><br><blockquote type="cite">you basically just need to attach it under<br>org.jboss.as.weld.deployment.WeldAttachments#PORTABLE_EXTENSIONS for<br>the top level deployment.<br></blockquote>Again, I can't do this without access to<br>WeldAttachments.PORTABLE_EXTENSIONS.<br><br>To break this cycle, I think I would need to create a new maven module<br>and subsytem called, for example, transactions-cdi. This seems a bit<br>over-kill. Is there an alternative you can think of?<br><br>Thanks,<br><br>Paul.<br><br><br>On 14 May 2013, at 10:15, Stuart Douglas <sdouglas@redhat.com<br><mailto:sdouglas@redhat.com>> wrote:<br><br><blockquote type="cite">Registering it in a DUP in the transactions subsystem is fine.<br><br>Have a look in<br>org.jboss.as.weld.deployment.processors.WeldPortableExtensionProcessor#loadAttachments<br>for an example of what you need to do, you basically just need to<br>attach it under<br>org.jboss.as.weld.deployment.WeldAttachments#PORTABLE_EXTENSIONS for<br>the top level deployment.<br><br>You can do this for every deployment, if beans.xml is not present it<br>won't do anything.<br><br>Stuart<br><br>Paul Robinson wrote:<br><blockquote type="cite">Stuart (wildfly-dev in CC too)<br><br>I'm implementing JTA 1.2, and part of it is to develop a new CDI<br>context. I'm currently registering that via an Extension in the<br>javax.enterprise.inject.spi.Extension file. The problem with this is<br>that this is a burden on the application developer. Alternatively, I<br>could register it in a DUP in the transactions subsytem. I'd probably<br>have to register it for every deployment, as I don't think I can detect<br>if it's needed.<br><br>@pmuir said that you would probably know what I should do.<br><br>Thanks,<br><br>Paul.<br>--<br>Paul Robinson<br>Web Service Transactions Lead<br>paul.robinson@redhat.com <mailto:paul.robinson@redhat.com><br><mailto:paul.robinson@redhat.com><br><br>JBoss, a Division of Red Hat<br>Registered in England and Wales under Company Registration No. 03798903<br>Directors: Michael Cunningham (USA), Brendan Lane (Ireland), Matt Parson<br>(USA), Charlie Peters (USA)<br><br></blockquote></blockquote><br>--<br>Paul Robinson<br>Web Service Transactions Lead<br>paul.robinson@redhat.com <mailto:paul.robinson@redhat.com><br><br>JBoss, a Division of Red Hat<br>Registered in England and Wales under Company Registration No. 03798903<br>Directors: Michael Cunningham (USA), Brendan Lane (Ireland), Matt Parson<br>(USA), Charlie Peters (USA)<br><br>_______________________________________________<br>wildfly-dev mailing list<br>wildfly-dev@lists.jboss.org<br>https://lists.jboss.org/mailman/listinfo/wildfly-dev<br></blockquote></blockquote></div><br><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">-- <br>Paul Robinson<br>Web Service Transactions Lead<br><a href="mailto:paul.robinson@redhat.com">paul.robinson@redhat.com</a><br><br>JBoss, a Division of Red Hat<br>Registered in England and Wales under Company Registration No. 03798903<br>Directors: Michael Cunningham (USA), Brendan Lane (Ireland), Matt Parson<br>(USA), Charlie Peters (USA)</div></span></span>
</div>
<br></div></div></body></html>