Tiles
  1. Tiles
  2. TILES-518

tiles-el should make org.apache.el an optional OSGi Import-Package

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.2.3
    • Component/s: tiles-el
    • Labels:
      None

      Description

      The tiles-el JAR file has an OSGi Import-Package entry for "org.apache.el". This is only necessary if one uses the TomcatExpressionFactoryFactory class, so the import should be optional. This can be achieved by adding the following plugin to the tiles-el POM file:

      <plugin>
      <groupId>org.apache.felix</groupId>
      <artifactId>maven-bundle-plugin</artifactId>
      <configuration>
      <instructions>
      <Import-Package>
      org.apache.el; resolution:=optional,
      *
      </Import-Package>
      </instructions>
      </configuration>
      </plugin>

        Activity

        Hide
        Antonio Petrelli added a comment -

        This bug is related to a Felix Maven Bundle plugin:
        https://issues.apache.org/jira/browse/FELIX-954
        However, until it is fixed, we need to change configuration.

        David, can you create and attach a patch?
        Thanks

        Show
        Antonio Petrelli added a comment - This bug is related to a Felix Maven Bundle plugin: https://issues.apache.org/jira/browse/FELIX-954 However, until it is fixed, we need to change configuration. David, can you create and attach a patch? Thanks
        Hide
        David Hay added a comment -

        Attaching patch for POM file in tiles-el module.

        Show
        David Hay added a comment - Attaching patch for POM file in tiles-el module.
        Hide
        Antonio Petrelli added a comment -

        Applied patch by David Hay. Thanks David!
        Anyway the next time, do a svn diff, since the name of the diff-ed file (pom.xml) was not there, instead I saw a "tiles-el.pom" unexisting file.

        Show
        Antonio Petrelli added a comment - Applied patch by David Hay. Thanks David! Anyway the next time, do a svn diff, since the name of the diff-ed file (pom.xml) was not there, instead I saw a "tiles-el.pom" unexisting file.

          People

          • Assignee:
            Antonio Petrelli
            Reporter:
            David Hay
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development