[jira] [Commented] (SLING-4906) Reduce (transitive) package dependencies

["Chetan Mehrotra (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SLING-4906?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14642324#comment-14642324 ] 

Chetan Mehrotra commented on SLING-4906:
----------------------------------------

Below is the current import list
{noformat}
  ch.qos.logback.classic                {version=[1.1,2)}
  ch.qos.logback.classic.encoder        {version=[1.1,2)}
  ch.qos.logback.classic.gaffer         {version=[1.1,2)}
  ch.qos.logback.classic.joran          {version=[1.1,2)}
  ch.qos.logback.classic.jul            {version=[1.1,2)}
  ch.qos.logback.classic.spi            {version=[1.1,2)}
  ch.qos.logback.classic.turbo          {version=[1.1,2)}
  ch.qos.logback.classic.util           {version=[1.1,2)}
  ch.qos.logback.core                   {version=[1.1,2)}
  ch.qos.logback.core.encoder           {version=[1.1,2)}
  ch.qos.logback.core.filter            {version=[1.1,2)}
  ch.qos.logback.core.helpers           {version=[1.1,2)}
  ch.qos.logback.core.joran             {version=[1.1,2)}
  ch.qos.logback.core.joran.action      {version=[1.1,2)}
  ch.qos.logback.core.joran.event       {version=[1.1,2)}
  ch.qos.logback.core.joran.spi         {version=[1.1,2)}
  ch.qos.logback.core.pattern           {version=[1.1,2)}
  ch.qos.logback.core.rolling           {version=[1.1,2)}
  ch.qos.logback.core.spi               {version=[1.1,2)}
  ch.qos.logback.core.status            {version=[1.1,2)}
  ch.qos.logback.core.util              {version=[1.1,2)}
  groovy.lang                           {resolution:=optional}
  javax.jms                             {resolution:=optional}
  javax.mail                            {resolution:=optional}
  javax.mail.internet                   {resolution:=optional}
  javax.management                      {resolution:=optional}
  javax.naming                          {resolution:=optional}
  javax.net                             
  javax.net.ssl                         
  javax.sql                             {resolution:=optional}
  javax.xml.namespace                   
  javax.xml.parsers                     {resolution:=optional}
  javax.xml.stream                      
  javax.xml.stream.events               
  org.codehaus.commons.compiler         {resolution:=optional}
  org.codehaus.groovy.control           {resolution:=optional}
  org.codehaus.groovy.control.customizers{resolution:=optional}
  org.codehaus.groovy.reflection        {resolution:=optional}
  org.codehaus.groovy.runtime           {resolution:=optional}
  org.codehaus.groovy.runtime.callsite  {resolution:=optional}
  org.codehaus.groovy.runtime.typehandling{resolution:=optional}
  org.codehaus.groovy.runtime.wrappers  {resolution:=optional}
  org.codehaus.janino                   {resolution:=optional}
  org.osgi.framework                    {version=1.3}
  org.osgi.util.tracker                 {version=[1.4,2)}
  org.slf4j                             {version=[1.6,1.8)}
  org.slf4j.helpers                     {version=[1.6,2)}
  org.slf4j.spi                         {version=[1.6,1.8)}
  org.xml.sax                           {resolution:=optional}
  org.xml.sax.helpers                   {resolution:=optional}
  sun.reflect                           {resolution:=optional}
{noformat}

So based on above we should not have imports for as others are system packages so safe to have imports
* groovy.lang  
* org.codehaus.groovy.*

> Reduce (transitive) package dependencies
> ----------------------------------------
>
>                 Key: SLING-4906
>                 URL: https://issues.apache.org/jira/browse/SLING-4906
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>            Reporter: Carsten Ziegeler
>            Assignee: Chetan Mehrotra
>             Fix For: Commons Log 4.0.4
>
>
> It seems that updating innocent looking bundles cause the whole system to restart. We now had a case where a bundle containing some package was updated. This package was imported by the groovy bundle and the groovy bundle was imported by the commons.log bundle.
> Unfortunately, the log4j.api bundle - which is imported by nearly almost every bundle - imports the impl package which is exported by the commons.log bundle.
> Obviously the best solution would be if the log4j.api bundle would not import packages, but I guess that's not feasible the way log4j works.
> Therefore we should have a look at our commons.log bundle to see whether we can reduce the imports, or the effect of other bundles being updated



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)