Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314760 - Provide connection / transport framework as API
Summary: Provide connection / transport framework as API
Status: RESOLVED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: Debug (show other bugs)
Version: 3.2   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.3   Edit
Assignee: Michael Rennie CLA
QA Contact: Simon Kaegi CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-27 15:34 EDT by Michael Rennie CLA
Modified: 2010-07-20 15:30 EDT (History)
1 user (show)

See Also:


Attachments
new transport project (39.84 KB, application/octet-stream)
2010-06-30 14:51 EDT, Michael Rennie CLA
no flags Details
update for the Crossfire bundle (124.94 KB, patch)
2010-06-30 14:52 EDT, Michael Rennie CLA
no flags Details | Diff
update for Rhino (170.92 KB, patch)
2010-06-30 15:41 EDT, Michael Rennie CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2010-05-27 15:34:23 EDT
code from HEAD

We have a lot of code in o.e.w.j.d.rhino.debugger.transport that is globally useful for anyone that wants to contribute a JSDI implementation + launching.

For example the experimental crossfire support had to copy all of those classes verbatim to be able to connect over a socket. In 3.3 we should provide an org.eclipse.wst.debug.transport (or similarly named) bundle so this code can be resued.
Comment 1 Michael Rennie CLA 2010-06-30 14:51:21 EDT
Created attachment 173144 [details]
new transport project

This is the entire new project: org.eclipse.wst.jsdt.debug.transport
Comment 2 Michael Rennie CLA 2010-06-30 14:52:18 EDT
Created attachment 173145 [details]
update for the Crossfire bundle

This patch updates the crossfire bundle to use the new API
Comment 3 Michael Rennie CLA 2010-06-30 15:41:07 EDT
Created attachment 173152 [details]
update for Rhino

This patch updates all of the Rhino bundles and the test suite
Comment 4 Michael Rennie CLA 2010-06-30 15:43:04 EDT
Simon, have a look at the first cut of the API and see what you think.
Comment 5 Simon Kaegi CLA 2010-07-20 12:50:29 EDT
The direction is right and a quick review didn't reveal anything that surprised me very much. The new transport bundle is a bit restricted and must run in non-OSGi environments because it will be used on the debugger side when folk want to debug a standalone rhino instance.

I think you're fine to check your changes in and we can work it into shape if needed.
Comment 6 Michael Rennie CLA 2010-07-20 15:30:52 EDT
applied patches to HEAD with minor updates:

1. removed all reference to core.runtime from the new transport bundle as it needs to be usable without eclipse at all.

2. updated some doc

Also updated the jsdt.feature and the map file for the new bundle, which has initially been tagged as v201007201421

Marking this bug fixed. Any subsequent updates / bugs can be in their own bug reports.