Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 255479 - [discovery][ui] org.eclipse.ecf.discovery.ui requires org.eclipse.ecf.remoteservices
Summary: [discovery][ui] org.eclipse.ecf.discovery.ui requires org.eclipse.ecf.remotes...
Status: RESOLVED FIXED
Alias: None
Product: ECF
Classification: RT
Component: ecf.ui (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Markus Kuppe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 256603
Blocks:
  Show dependency tree
 
Reported: 2008-11-17 02:45 EST by Remy Suen CLA
Modified: 2008-12-12 03:22 EST (History)
2 users (show)

See Also:

Attachments
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 Remy Suen CLA 2008-11-17 02:45:14 EST 
Quite simply, the discovery UI bundle should not require the remoteservices API. For some reasons, the AbstractRemoteServiceAccessHandler class is in the discovery UI bundle, which appears to have introduced this requirement.
Comment 1 Scott Lewis CLA 2008-11-17 04:52:24 EST 
(In reply to comment #0)
> Quite simply, the discovery UI bundle should not require the remoteservices
> API. For some reasons, the AbstractRemoteServiceAccessHandler class is in the
> discovery UI bundle, which appears to have introduced this requirement.
> 

Yes...the AbstractRemoteServiceAccessHandler class should be moved out of this (discovery ui) bundle to eliminate this dep.  It probably makes sense to move to the o.e.e.remoteservices.ui bundle...and this would introduce a dep to o.e.e.discovery.ui I believe...which seems OK to me.

All on 3.0 stream (HEAD) though.
Comment 2 Remy Suen CLA 2008-11-27 07:00:51 EST 
This is Markus's problem now. :)
Comment 3 Markus Kuppe CLA 2008-12-12 03:22:55 EST 
Fixed in HEAD with 256603