Bug 226764 - [rseterminal] Integrate the TM Terminal View with RSE
Summary: [rseterminal] Integrate the TM Terminal View with RSE
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P2 enhancement (vote)
Target Milestone: 3.0 M7   Edit
Assignee: Anna Dushistova CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on: 170910
Blocks: 226550 227535 227537 227539 227569 227571 227572 229479
  Show dependency tree
 
Reported: 2008-04-11 17:05 EDT by Martin Oberhuber CLA
Modified: 2008-08-19 12:51 EDT (History)
8 users (show)

See Also:


Attachments
Plugins for RSE Terminal Integration (52.57 KB, application/zip)
2008-04-11 17:05 EDT, Martin Oberhuber CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Oberhuber CLA 2008-04-11 17:05:34 EDT
Created attachment 95759 [details]
Plugins for RSE Terminal Integration

+++ This bug was initially created as a clone of Bug #170910 +++

Based on the new Terminal APIs from bug 170910, MontaVista's contribution of an initial RSE Terminal Integration should be added as a separate RSE Feature in Incubation to start with.

Attached is the current contribution which builds against RSE 3.0M6 candidate. It's still got several problems, such as
  * On disconnect, "connected" status is not revoked
  * Terminal Encodings should be set (pending bug 204796)
  * "Launch Terminal" action in file subsystem should be contributed
    declaratively (this is bug 226550)
  * Contribution should be re-written such that it can take an existing
    RSE IShellService and ask for getAdapter(ITerminalService) in a generic
    way. This would help such that only ONE subsysteConfiguration is needed
    that can run against ANY IShellServiceSubsystem. Problem with that 
    approach, though, might be that the "Remote Command" subsystem cannot
    be hidden and entirely replaced by Terminal Subsystem in that case
  * In the case where ONLY Terminal subsystem exists but NO shell subsystem,
    adaptable mechanism should be used in order to still provide access
    to the "Linux Shell Processes Subsystem" which requires an IHostShell

Original contribution was submitted with legal message as per bug 170910 comment 17:

We, Anna Dushistova,Yu-Fen Kuo and Eugeniy Melekhov, declare that we developed
attached code from scratch, without referencing any 3rd party materials except
material licensed under the EPL. We are authorized by our employer to
make this contribution under the EPL.
Comment 1 Martin Oberhuber CLA 2008-04-11 22:04:06 EDT
I submitted the contribution on IPZilla for Legal Review:
https://dev.eclipse.org/ipzilla/show_bug.cgi?id=2256
Comment 2 Martin Oberhuber CLA 2008-04-16 17:02:38 EDT
The contribution was approved, and I checked in the three plugins under

   /cvsroot/dsdp , org.eclipse.tm.rse/plugins/

I'm keeping this bug open in order to add the also required changes for Release Engineering, such as defining a feature, modifying the team project set, and adding it to the Downloads page.

I'd also like to see more contributions for fixing some of the urgent issues with the current code. We'll see if we keep everything on this bug or split it up into separate bug reports as needed.
Comment 3 Martin Oberhuber CLA 2008-04-16 17:05:48 EDT
I'm wondering what code in this contribution specifically comes from Eugeniy Melekhov. He is not mentioned in any of the Copyright Headers. Do I need to add him to the list of contributors for DSDP-TM?
Comment 4 Anna Dushistova CLA 2008-04-16 17:46:21 EDT
Martin, Eugeniy participated in API discussions and new API creation only.
Comment 5 Anna Dushistova CLA 2008-04-17 05:38:08 EDT
Is the expected behaviour on disconnect the same as for shells subsystem?
Comment 6 Martin Oberhuber CLA 2008-04-17 07:14:38 EDT
Unless you can propose something else, I'd say yes?
Comment 7 Martin Oberhuber CLA 2008-04-17 11:18:55 EDT
The Terminal Subsystem has been checked in, and added to releng such that it's available as a download as well as on the update site:

http://download.eclipse.org/dsdp/tm/downloads/drops/I20080417-0800/index.php

I'm marking this bug fixed, we'll track the remaining issues in separate bugs, all of which are available from the bugzilla "blocks" field for now.