Uploaded image for project: 'MyFaces Core'
  1. MyFaces Core
  2. MYFACES-2976

Support hiding myfaces impl classes in osgi, and provide a single osgi bundle for api + impl

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.0.3-SNAPSHOT
    • 2.0.3
    • Extension Feature
    • None

    Description

      The current osgi solution of using require-bundle between the api and impl jars is generally thought of as osgi worst-practice and just doesn't work with snapshot versions (presumably a bug in the bundle plugin).

      Also this does not easily support hiding the impl classes.

      Geronimo would like to use a single bundle containing both api and impl and hide the impl classes as much as possible. This requires some slight modifications to the FactoryFinder so that impl classes can be loaded from the same classloader as FactoryFinder if they are not visible to the context class loader.

      I think myfaces would be the best place for this combined bundle but geronimo can build it if necessary. The FactoryFinder classloader changes are therefore more important.

      I've suggested a module name which maps to the bundle symbolic name well, as this seems to be the preferred maven/osgi strategy at apache.

      Attachments

        1. MYFACES-2976-osgi-bundle.patch
          7 kB
          Jakob Korherr
        2. MYFACES-2976.diff
          10 kB
          David Jencks

        Issue Links

          Activity

            People

              jakobkorherr Jakob Korherr
              djencks David Jencks
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: