Bugzilla – Full Text Bug Listing
| Summary: | [Gogo] obr commands should not be present in Virgo | ||
|---|---|---|---|
| Product: | [RT] Virgo | Reporter: | Glyn Normington <glyn.normington> |
| Component: | runtime | Assignee: | Project Inbox <virgo-inbox> |
| Status: | CLOSED WONTFIX | QA Contact: | |
| Severity: | minor | ||
| Priority: | P3 | CC: | b.kapukaranov, gunnar, l.kirchev |
| Version: | 3.0.0.M06 | ||
| Target Milestone: | 3.5.0.RELEASE | ||
| Hardware: | PC | ||
| OS: | Mac OS X - Carbon (unsup.) | ||
| Whiteboard: | |||
|
Description
Glyn Normington
(In reply to comment #0) > As the OBR spec is in flux and Virgo is making no attempt to add OBR support to > p2, I think we should remove these commands as they will only prompt users to > try integrating Felix OBR support which will result in much angst. Gogo registers the OBR commands by default. Should see how we can fix this. The Gogo shell bundle registers these commands by default. Actually, they do not work, because there is no OBR repository. But we cannot unregister them either - it is not possible to unregister a service from a bundle, different from the one which registered it. Is this a big problem? It's not a big problem, which is why this bug is classified as minor. However, I don't think the Gogo bundle should register these commands by default, especially if they do not work. This is the kind of behaviour you would expect of monolithic software. I think OBR should provide and register the commands rather than having them registered by Gogo. We have no plans to implement the OBR spec (once it has been standardised) in Virgo, so I would like a way of avoiding these commands. Perhaps you could raise a bug against Gogo and link it up with this one? (In reply to comment #3) > It's not a big problem, which is why this bug is classified as minor. > > However, I don't think the Gogo bundle should register these commands by > default, especially if they do not work. This is the kind of behaviour you > would expect of monolithic software. I think OBR should provide and register > the commands rather than having them registered by Gogo. > > We have no plans to implement the OBR spec (once it has been standardised) in > Virgo, so I would like a way of avoiding these commands. > > Perhaps you could raise a bug against Gogo and link it up with this one? Yes, I will open a bug - this seems to be the only possible thing we can do if we do not want to fork the Gogo project. Thanks very much Lazar. We certainly do not want to fork Gogo. Since we cannot directly unregister the commands, we will embed the Gogo command bundle in our own bundle, which will register only those commands from Gogo command, which we use. Thus we will have a new bundle on the place of Gogo command bundle. Unfortunately, I think we have to live with this. (In reply to comment #7) > Unfortunately, I think we have to live with this. Just to note that Tom Watson opened a bug against Felix to move the help command from the commands bundle to runtime bundle, so that we can drop the commands bundle at all. If this happens the problem with the OBR commands will get solved automatically. (In reply to comment #8) Thanks Lazar! (In reply to comment #8) > (In reply to comment #7) > > Unfortunately, I think we have to live with this. > > Just to note that Tom Watson opened a bug against Felix to move the help > command from the commands bundle to runtime bundle, so that we can drop the > commands bundle at all. If this happens the problem with the OBR commands will > get solved automatically. For the record, the bug is https://issues.apache.org/jira/browse/FELIX-2340 |