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

        David Hay created issue -
        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.
        David Hay made changes -
        Field Original Value New Value
        Attachment tiles-el.pom.patch [ 12464963 ]
        Antonio Petrelli made changes -
        Original Estimate 0h [ 0 ]
        Remaining Estimate 0h [ 0 ]
        Assignee Antonio Petrelli [ brenmcguire ]
        Antonio Petrelli made changes -
        Fix Version/s 2.2.3 [ 12315143 ]
        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.
        Antonio Petrelli made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        5d 20h 42m 1 Antonio Petrelli 05/Dec/10 19:34

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development