Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325384 - Request StateFolder vom OMPlatform fails while runing inside an OSGiFramework
Summary: Request StateFolder vom OMPlatform fails while runing inside an OSGiFramework
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.net4j (show other bugs)
Version: 4.0   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Eike Stepper CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-15 15:07 EDT by Heiko Ahlig CLA
Modified: 2011-06-23 03:37 EDT (History)
0 users

See Also:

Attachments
Patch-V1 (1.24 KB, patch)
2010-09-15 15:12 EDT, Heiko Ahlig CLA 		stepper: iplog+
Details | Diff
Patch v2 (2.67 KB, patch)
2010-09-23 11:54 EDT, Eike Stepper CLA 		no flags Details | Diff
Show Obsolete (1) 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 Heiko Ahlig CLA 2010-09-15 15:07:29 EDT 
Build Identifier: 

If you request the OMPlatform.INSTANCE.getStateFolder() inside an OSGIApplication, and net4j.state  is not set it falls back to the osgi.instance.area property.
This property give back a URI like String. Build a file from this plain String it fails because it generates wrong file like:
E:\eclipse\file:\E:\runtime-de.nacorswelt.single.application\.metadata 
 

Reproducible: Always
Comment 1 Heiko Ahlig CLA 2010-09-15 15:12:36 EDT 
Created attachment 178974 [details]
Patch-V1
Comment 2 Eike Stepper CLA 2010-09-23 11:54:02 EDT 
Hi Heiko, when you attach a patch it's easier for me if you make that patch a "workspace relative patch".

I've slightly changed your patch. Can you review it and tell me what you think?
Comment 3 Eike Stepper CLA 2010-09-23 11:54:28 EDT 
Created attachment 179469 [details]
Patch v2
Comment 4 Heiko Ahlig CLA 2010-09-24 13:44:53 EDT 
I have reviewed your patch and he works looks better than my patch.

For Patch V1:
It is smaller than 250 lines, I'm the owner and writer of the patch and I apply it to the EPL.
Comment 5 Eike Stepper CLA 2010-09-24 13:46:50 EDT 
Thank you!

Committed to HEAD
Comment 6 Eike Stepper CLA 2011-06-23 03:37:39 EDT 
Available in R20110608-1407