[ https://issues.apache.org/jira/browse/SLING-3814?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14084448#comment-14084448 ]
Chetan Mehrotra commented on SLING-3814: ---------------------------------------- bq. Do you really need to embed the groovy-all dependency? IIRC installing groovy-all is enough to activate Groovy scripting in general, so why not just expect it to be present? The Groovy dependency would then need to be resolved at Start Level 1 which would cause issue with existing setup where Groovy would be running at higher level. Further embedding Groovy would also isolate logging dependencies from others as Groovy bundle has a Dynamic-Import for * which causes it to be refreshed often. > Groovy fragment bundle for Common Log to support scriptable event evaluation > ---------------------------------------------------------------------------- > > Key: SLING-3814 > URL: https://issues.apache.org/jira/browse/SLING-3814 > Project: Sling > Issue Type: New Feature > Components: Extensions > Reporter: Chetan Mehrotra > Assignee: Chetan Mehrotra > Attachments: > org.apache.sling.extensions.logback-groovy-fragment-1.0.0-SNAPSHOT-src.zip > > > Logback provides support for scriptable event evaluation [1] where a script > can be used to determine if the given logging event must be logged or not. > This proves useful to get precise logs in a concurrent env. For this Logback > support > * Janino - Allow writing Java Code and is faster compared to Groovy. Due to > [LOGBACK-1001|http://jira.qos.ch/browse/LOGBACK-1001] it cannot be used > currently > * Groovy > Commons Log bundle can have optional dependency on Groovy classes but that > would cause issues as Commons Log starts at level 1 and further for stability > of Logging system it would be preferable if it gets its own copy of Groovy > (via a fragement). > This feature is mostly to be used for debugging issues > [1] http://logback.qos.ch/manual/filters.html -- This message was sent by Atlassian JIRA (v6.2#6252)