Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364433 - IRemoteProxyManager needs ability to use a URI to manufacture an IRemoteFileProxy and IRemoteCommandLauncher
Summary: IRemoteProxyManager needs ability to use a URI to manufacture an IRemoteFileP...
Status: RESOLVED FIXED
Alias: None
Product: Linux Tools
Classification: Tools
Component: Project (show other bugs)
Version: 0.9.0   Edit
Hardware: All Linux
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Jeff Johnston CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 364426
Blocks: 365381
  Show dependency tree
 
Reported: 2011-11-21 21:10 EST by Corey Ashford CLA
Modified: 2012-05-29 13:14 EDT (History)
1 user (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 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.