Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 255715 - EquinoxLauncher arbitrarily sets osgi.framework.activeThreadType
Summary: EquinoxLauncher arbitrarily sets osgi.framework.activeThreadType
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Framework (show other bugs)
Version: 3.5   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.5 M4   Edit
Assignee: equinox.framework-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-18 16:24 EST by Erin Schnabel CLA
Modified: 2008-11-19 13:02 EST (History)
1 user (show)

See Also:

Attachments
patch for checking for the set property first (1.44 KB, patch)
2008-11-19 10:23 EST, Erin Schnabel CLA 		tjwatson: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Erin Schnabel CLA 2008-11-18 16:24:16 EST 
When using the new EquinoxLauncher, the first line of code in the setEquinoxProperties method arbitrarily sets the "osgi.framework.activeThreadType" to "normal"...

Since the constructor of the framework could conceivably wait/listen for shutdown itself, it isn't necessarily true that the framework has to create a non-daemon thread to prevent the JVM from shutting down.

Need to check for alternate values of this property before just setting this one.

Tom can correct my phrasing... Patch attached.
Comment 1 Thomas Watson CLA 2008-11-19 09:38:37 EST 
(In reply to comment #0)
> Tom can correct my phrasing... Patch attached.
> 

Your phrasing seems correct, but your patch is not attached :)

Please attach your patch, thanks.
Comment 2 Erin Schnabel CLA 2008-11-19 10:23:02 EST 
Created attachment 118260 [details]
patch for checking for the set property first

Weird. Must not like patches attached when the bug is opened?
Comment 3 Thomas Watson CLA 2008-11-19 13:02:03 EST 
Comment on attachment 118260 [details]
patch for checking for the set property first

Thanks Erin.  Adding iplog flag.
Comment 4 Thomas Watson CLA 2008-11-19 13:02:34 EST 
Patch released.