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

Jean-Baptiste Onofré commented on FELIX-6034:
---------------------------------------------

[~rotty3000] yes, it's what I did in Karaf shell core, and it works fine.

However I didn't get the purpose of the requirement for two reasons:

1. Karaf shell.core provides the local shell console (based on jline) and wrap 
gogo.jline for the Posix commands. It doesn't provide other commands than the 
core one.
2. Karaf shell.core starts in first place before other commands, so, the 
requirement seems too strict to me as it can start "empty" without any command 
yet.

That's why I asked ;)

> Gogo JLine requirement doesn't allow to embed gogo.jline
> --------------------------------------------------------
>
>                 Key: FELIX-6034
>                 URL: https://issues.apache.org/jira/browse/FELIX-6034
>             Project: Felix
>          Issue Type: Bug
>          Components: Gogo JLine
>    Affects Versions: gogo.jline-1.1.2
>            Reporter: Jean-Baptiste Onofré
>            Assignee: Jean-Baptiste Onofré
>            Priority: Major
>             Fix For: gogo.jline-1.1.3
>
>
> Felix Gogo Jline 1.1.2 introduced a Requirement:
> {code}
> @Requirement(
>     effective = "active",
>     namespace = "org.apache.felix.gogo",
>     name = "command.implementation",
>     version = "1.0.0"
> )
> {code}
> This requirement is a contract to find concrete command. However, in the case 
> of Gogo JLine embedded as a standalone bundle, waiting for commands 
> implementation (later on), this requirement doesn't allow to start. That's 
> the case in Apache Karaf shell.
> [~rotty3000] do you mind if I remove this requirement (as I did other fixes 
> like in {{Posix}} commands) ? We could add this Requirement in gogo.jline 2.x 
> and then, I will have to find a bypass or at least a fake command providing a 
> capability.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to