Uploaded image for project: 'Solder'
  1. Solder
  2. SOLDER-47

Cannot use Solder in Glassfish 3.1

        Details

        • Steps to Reproduce:
          Hide

          Download the seam-catch-example-jaxrs.war and try to deploy it into a glassfish 3.1 build.

          Show
          Download the seam-catch-example-jaxrs.war and try to deploy it into a glassfish 3.1 build.

          Description

          Please see http://java.net/jira/browse/GLASSFISH-14808 for additional information I communicated to Siva.

          Siva has closed this bug saying it's not a bug because weld-extensions.jar doesn't have a beans.xml file, which isn't valid, it doesn't need one. You can see the exceptions I've been fighting trying to get my catch jaxrs example running in glassfish 3.1

            Gliffy Diagrams

              Activity

              Ascending order - Click to sort in descending order
              4 older comments
              Hide
              Permalink
              dan.j.allen Dan Allen added a comment -

              There's an important detail left out in the explanation here. Seam Solder registers it's own beans manually using the BeanManager#addAnnotatedType() API. See ELExtension [1] for an example. That's one reason why it doesn't require a beans.xml. This is the spec way of registering classes from third-party libraries as beans.

              Also, the reason beans.xml isn't require in an extension library is because extensions are loaded as META-INF/services, which makes no mention (nor has no prerequisite) on a beans.xml file. This is a standard part of Java 6 (which can be provided for Java 5 by a spec implementation, as Weld does).

              [1] https://github.com/seam/solder/blob/master/impl/src/main/java/org/jboss/seam/solder/el/ELExtension.java

              Show
              dan.j.allen Dan Allen added a comment - There's an important detail left out in the explanation here. Seam Solder registers it's own beans manually using the BeanManager#addAnnotatedType() API. See ELExtension [1] for an example. That's one reason why it doesn't require a beans.xml. This is the spec way of registering classes from third-party libraries as beans. Also, the reason beans.xml isn't require in an extension library is because extensions are loaded as META-INF/services, which makes no mention (nor has no prerequisite) on a beans.xml file. This is a standard part of Java 6 (which can be provided for Java 5 by a spec implementation, as Weld does). [1] https://github.com/seam/solder/blob/master/impl/src/main/java/org/jboss/seam/solder/el/ELExtension.java
              Hide
              Permalink
              jharting Jozef Hartinger added a comment -

              FYI, http://java.net/jira/browse/GLASSFISH-14808 has been reopened and the issue seems to be fixed. Track GLASSFISH-14808 to see when the fix gets into a release.

              Show
              jharting Jozef Hartinger added a comment - FYI, http://java.net/jira/browse/GLASSFISH-14808 has been reopened and the issue seems to be fixed. Track GLASSFISH-14808 to see when the fix gets into a release.
              Hide
              Permalink
              dan.j.allen Dan Allen added a comment -

              Here is the commit that we've been waiting for in GlassFish:

              http://java.net/projects/glassfish/sources/svn/revision/44294

              We should create an Arquillian test in Solder that validates the behavior of extensions in non-bean archives and then push that test out to the CDI TCK.

              Show
              dan.j.allen Dan Allen added a comment - Here is the commit that we've been waiting for in GlassFish: http://java.net/projects/glassfish/sources/svn/revision/44294 We should create an Arquillian test in Solder that validates the behavior of extensions in non-bean archives and then push that test out to the CDI TCK.
              Hide
              Permalink
              dan.j.allen Dan Allen added a comment -

              Here's the Hudson build for GlassFish so we can test this out: http://hudson.glassfish.org/job/gf-trunk-build-dev/

              Show
              dan.j.allen Dan Allen added a comment - Here's the Hudson build for GlassFish so we can test this out: http://hudson.glassfish.org/job/gf-trunk-build-dev/
              Hide
              Permalink
              shane.bryzak Shane Bryzak added a comment -

              I believe this has been addressed now in Glassfish, however I'm going to bump this issue until it can be confirmed.

              Show
              shane.bryzak Shane Bryzak added a comment - I believe this has been addressed now in Glassfish, however I'm going to bump this issue until it can be confirmed.

                People

                • Assignee:
                  shane.bryzak Shane Bryzak
                  Reporter:
                  lightguard Jason Porter
                • Votes:
                  3 Vote for this issue
                  Watchers:
                  8 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development