You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tiles.apache.org by "Antonio Petrelli (JIRA)" <ji...@apache.org> on 2010/12/05 20:35:12 UTC

[jira] Commented: (TILES-518) tiles-el should make org.apache.el an optional OSGi Import-Package

    [ https://issues.apache.org/jira/browse/TILES-518?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12967008#action_12967008 ] 

Antonio Petrelli commented on TILES-518:
----------------------------------------

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.

> tiles-el should make org.apache.el an optional OSGi Import-Package
> ------------------------------------------------------------------
>
>                 Key: TILES-518
>                 URL: https://issues.apache.org/jira/browse/TILES-518
>             Project: Tiles
>          Issue Type: Bug
>          Components: tiles-el
>    Affects Versions: 2.2.2
>            Reporter: David Hay
>            Assignee: Antonio Petrelli
>             Fix For: 2.2.3
>
>         Attachments: tiles-el.pom.patch
>
>   Original Estimate: 0h
>  Remaining Estimate: 0h
>
> 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>

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.