Bug 239007

Summary: [discovery][jslp] Remove DEBUG OUTPUT: false from log
Product: [RT] ECF Reporter: Scott Lewis <slewis>
Component: ecf.discoveryAssignee: Markus Kuppe <bugs.eclipse.org>
Status: RESOLVED FIXED QA Contact:
Severity: minor    
Priority: P3 CC: bugs.eclipse.org
Version: 2.0.0   
Target Milestone: 2.0.1   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 244678    
Bug Blocks:    

Description Scott Lewis CLA 2008-06-30 11:23:45 EDT 
In OSGiPlatformAbstraction line 84 is a log warning sent to the log service.  When starting in Eclipse, this produces output 

DEBUG OUTPUT: false 

to the Eclipse log when not debugging.  This should be removed or disabled so that by default no output is sent to the log when Eclipse is starting.
Comment 1 Markus Kuppe CLA 2008-07-01 02:40:44 EDT 
https://sourceforge.net/tracker/index.php?func=detail&aid=2007432&group_id=151721&atid=781930

Btw. if LogService is active during jSLP bundle startup, the stmt would be send there and not show up in the Eclipse Error log. ;)
Comment 2 Scott Lewis CLA 2008-07-01 02:53:22 EDT 
(In reply to comment #1)
> https://sourceforge.net/tracker/index.php?func=detail&aid=2007432&group_id=151721&atid=781930
> 
> Btw. if LogService is active during jSLP bundle startup, the stmt would be send
> there and not show up in the Eclipse Error log. ;)
> 

We don't have any control over the behavior of LogService in the Eclipse context...so I'm not sure what you are saying.
Comment 3 Markus Kuppe CLA 2008-07-01 03:37:40 EDT 
(In reply to comment #2)
> We don't have any control over the behavior of LogService in the Eclipse
> context...so I'm not sure what you are saying.

It's meant as a workaround until we remove the debug stmt from jSLP and get a new CQ in.
Comment 4 Markus Kuppe CLA 2008-07-01 03:38:56 EDT 
...in scenarios where one has control over log service, e.g. in a RCP.
Comment 5 Scott Lewis CLA 2008-07-01 10:54:51 EDT 
Setting target milestone and assigning to Markus.
Comment 6 Markus Kuppe CLA 2008-07-15 02:47:49 EDT 
Fixed in upstream (r243). Will hit ECF with a new CQ for jSLP.
Comment 7 Markus Kuppe CLA 2008-08-22 02:26:18 EDT 
Fixed in orbit