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

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

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

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment