Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 364433

Summary: IRemoteProxyManager needs ability to use a URI to manufacture an IRemoteFileProxy and IRemoteCommandLauncher
Product: [Tools] Linux Tools Reporter: Corey Ashford <cjashfor>
Component: ProjectAssignee: Jeff Johnston <jjohnstn>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: linux.distros-inbox
Version: 0.9.0   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Bug Depends on: 364426    
Bug Blocks: 365381    

Description Corey Ashford CLA 2011-11-21 21:10:58 EST
Build Identifier: M20110909-1335

In order to modify the RemoteConnection class to be able to use the IRemoteFileProxy and IRemoteLaunch interfaces, we need the ability of IRemoteProxyManager to manufacture a proxy based on a URI, in addition to an IProject (which is being used by the remote Autotools project)


Reproducible: Always

Steps to Reproduce:
N/A
Comment 1 Corey Ashford CLA 2011-11-21 21:15:45 EST
The first comment says, "IRemoteLaunch interface", but should say "IRemoteCommandLauncher interface".
Comment 2 Corey Ashford CLA 2011-11-21 21:18:44 EST
I have pushed up a patch to add the capability of creating the two proxy objects
from a URI.  It is located here:

git@github.com:cjashfor/LinuxToolsProjectPatches.git

in the branch rdt_add_uri
Comment 3 Andrew Overholt CLA 2011-11-25 16:15:57 EST
Without a "remote" bucket, let's just assign this to Jeff and see what happens :)
Comment 4 Corey Ashford CLA 2011-12-01 18:33:43 EST
Jeff, have you had a chance to look at this patch?
Comment 5 Jeff Johnston CLA 2011-12-01 19:03:58 EST
(In reply to comment #4)
> Jeff, have you had a chance to look at this patch?

I will review it tomorrow.
Comment 6 Jeff Johnston CLA 2011-12-02 17:22:37 EST
(In reply to comment #4)
> Jeff, have you had a chance to look at this patch?

Patches applied to rdt branch.  I ran into a conflict in RDTProxyManager.java.  I made a subsequent commit that added ChangeLog entries for the two plug-ins being changed.
Comment 7 Corey Ashford CLA 2011-12-02 17:38:58 EST
(In reply to comment #6)
> (In reply to comment #4)
> > Jeff, have you had a chance to look at this patch?
> 
> Patches applied to rdt branch.  I ran into a conflict in RDTProxyManager.java. 
> I made a subsequent commit that added ChangeLog entries for the two plug-ins
> being changed.

Thanks, Jeff!
Comment 8 Corey Ashford CLA 2012-05-29 13:14:57 EDT
I wrote 100% of this patch, had permission to do so from my employer IBM Corp., and the code is EPL licensed.