Bug 484900 - CDT 9.0 Remove CDI Debug
Summary: CDT 9.0 Remove CDI Debug
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-cdi (show other bugs)
Version: 9.0.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 9.0.0   Edit
Assignee: Project Inbox CLA
QA Contact: Nobody - feel free to take it CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 566530
  Show dependency tree
 
Reported: 2015-12-25 18:54 EST by Jonah Graham CLA
Modified: 2020-08-31 13:47 EDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonah Graham CLA 2015-12-25 18:54:00 EST
(I can't find an existing bug on this)

One of the items on the 9.0 Planning [1] is removal of CDI Debug. This is the bugzilla to track that.



[1] https://wiki.eclipse.org/CDT/planning/9.0
Comment 1 Eclipse Genie CLA 2015-12-25 19:07:07 EST
New Gerrit change created: https://git.eclipse.org/r/63282
Comment 2 Jonah Graham CLA 2015-12-26 15:23:17 EST
Some methods and classes got marked for removal in https://git.eclipse.org/r/#/c/63288/
Comment 3 Doug Schaefer CLA 2015-12-26 15:28:18 EST
Very cool! Thanks, Jonah. I'll take a look once I get a few other things done. Might not happen until after the holidays.
Comment 4 Jonah Graham CLA 2015-12-26 15:32:12 EST
(In reply to Doug Schaefer from comment #3)
> Very cool! Thanks, Jonah. I'll take a look once I get a few other things
> done. Might not happen until after the holidays.

No problem. I know you are busy, just hoping I can help a little here. All of this has been motivated by exploring the code as a result of my work on Bug 472765. I'll keep at it, have a review when you get a chance. Happy holidays!
Comment 5 Doug Schaefer CLA 2015-12-26 15:43:01 EST
You've been a huge help! Much appreciated.

Happy holidays!
Comment 7 Jonah Graham CLA 2016-03-08 20:49:39 EST
(In reply to Eclipse Genie from comment #6)
> Gerrit change https://git.eclipse.org/r/63282 was merged to [master].
> Commit:
> http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/
> ?id=ff7f25b94d9d4978be6ebe32cc2b39d924de6062

Now that the "big" one has gone in there are a few outstanding issues:

 - Remove most of the public interfaces of CDI
(org.eclipse.cdt.debug.core.model) are still in place, many of them are
part of DSF so its not as simple as delete the whole package

- Activities need to be updated/reviewed

- The plugin.xmls have not been reviewed much, only so far to remove dependencies on classes that were removed.

- LocalCDILaunchDelegate should be removed, for now I have only removed
all the debug parts of it, but most of what is left delegates to
LocalRunLaunchDelegate

- Cleanup/Rename CDI* classes used by DSF
Comment 8 Eclipse Genie CLA 2016-03-08 21:03:30 EST
New Gerrit change created: https://git.eclipse.org/r/68011
Comment 9 Eclipse Genie CLA 2016-03-08 21:03:32 EST
New Gerrit change created: https://git.eclipse.org/r/68010
Comment 11 Jonah Graham CLA 2016-03-16 14:50:28 EDT
Commit http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=28b1830acf942216ae99d4d167b6c3b2133aaec6 added some deprecations that can be simply removed if I can get to them this week.

Lines are marked "@deprecated Only used by CDI, scheduled for removal as part of Bug 484900"
Comment 12 Eclipse Genie CLA 2016-03-16 19:11:35 EDT
New Gerrit change created: https://git.eclipse.org/r/68598
Comment 14 Eclipse Genie CLA 2016-03-18 11:43:06 EDT
New Gerrit change created: https://git.eclipse.org/r/68785
Comment 15 Eclipse Genie CLA 2016-03-18 12:40:23 EDT
New Gerrit change created: https://git.eclipse.org/r/68788
Comment 16 Eclipse Genie CLA 2016-03-18 13:20:40 EDT
New Gerrit change created: https://git.eclipse.org/r/68795
Comment 17 Eclipse Genie CLA 2016-03-18 14:16:17 EDT
New Gerrit change created: https://git.eclipse.org/r/68799
Comment 23 Eclipse Genie CLA 2016-03-18 22:36:58 EDT
New Gerrit change created: https://git.eclipse.org/r/68827
Comment 24 Marc Khouzam CLA 2016-04-05 16:24:05 EDT
(In reply to Eclipse Genie from comment #23)
> New Gerrit change created: https://git.eclipse.org/r/68827

I've abandoned this one.
Comment 25 Marc Khouzam CLA 2016-04-05 16:27:05 EDT
(In reply to Jonah Graham from comment #7)

> Now that the "big" one has gone in there are a few outstanding issues:
> 
>  - Remove most of the public interfaces of CDI
> (org.eclipse.cdt.debug.core.model) are still in place, many of them are
> part of DSF so its not as simple as delete the whole package
> 
> - Activities need to be updated/reviewed
> 
> - The plugin.xmls have not been reviewed much, only so far to remove
> dependencies on classes that were removed.
> 
> - LocalCDILaunchDelegate should be removed, for now I have only removed
> all the debug parts of it, but most of what is left delegates to
> LocalRunLaunchDelegate

This has been done.

> - Cleanup/Rename CDI* classes used by DSF

At EclipeCon2016 we agreed to keep old classes but make then stubs that point to a properly named set of classes.
Comment 26 Eclipse Genie CLA 2016-12-05 16:48:24 EST
New Gerrit change created: https://git.eclipse.org/r/86411
Comment 28 Alvaro Sanchez-Leon CLA 2016-12-09 08:59:54 EST
It seems the main job has been done and released, 

   shouldn't we marked this one as fixed now ?
Comment 29 Jonah Graham CLA 2016-12-09 09:17:35 EST
(In reply to Alvaro Sanchez-Leon from comment #28)
> It seems the main job has been done and released, 
> 
>    shouldn't we marked this one as fixed now ?
Yes!

Note I put "Fixed version" as 9.0. Everything except the recent documentation cleanup (https://git.eclipse.org/r/86411) was in 9.0, the documentation one is in 9.3

I created bug 508992 to cover any of the remnants.
Comment 30 Alvaro Sanchez-Leon CLA 2016-12-09 16:01:44 EST
(In reply to Jonah Graham from comment #29)

> Note I put "Fixed version" as 9.0. Everything except the recent
> documentation cleanup (https://git.eclipse.org/r/86411) was in 9.0, the
> documentation one is in 9.3
> 
> I created bug 508992 to cover any of the remnants.

Great ! Thanks Jonah !
Comment 31 Simeon Andreev CLA 2020-03-04 07:55:19 EST
Hi Jonah,

does any wiki page exist, that is dedicated to migrating code that used the removed APIs?

I'm guessing there was a paradigm switch, I would like to read about it; we are in the process of updating an older product to CDT 9.x, we have some code that uses the (apparently) old CDT debug framework.

Best regards and thanks,
Simeon
Comment 32 Jonah Graham CLA 2020-03-04 16:22:16 EST
(In reply to Simeon Andreev from comment #31)
> Hi Jonah,
> 
> does any wiki page exist, that is dedicated to migrating code that used the
> removed APIs?
> 
> I'm guessing there was a paradigm switch, I would like to read about it; we
> are in the process of updating an older product to CDT 9.x, we have some
> code that uses the (apparently) old CDT debug framework.
> 
> Best regards and thanks,
> Simeon

It is a paradigm switch - DSF came on the scene a few years before CDI was deprecated and eventually removed.

Intro to DSF: https://help.eclipse.org/latest/topic/org.eclipse.cdt.doc.isv/guide/dsf/intro/dsf_programming_intro.html?cp=14_0_5

An example of an extension to DSF-GDB that shows some examples of how you can extend to add additional functionality. The main part of the example shows how to add a "Show GDB Version" command in various places in the UI and how to tie it in to the backend: https://git.eclipse.org/c/cdt/org.eclipse.cdt.git/tree/dsf-gdb/org.eclipse.cdt.examples.dsf.gdb

In the past I have done training for companies on DSF migration. I haven't touched it recently, but the github repo is still in place: https://github.com/jonahgraham/cdt-examples

I added the presentation that goes with cdt-examples to the repo.
Comment 33 Simeon Andreev CLA 2020-03-05 03:01:30 EST
(In reply to Jonah Graham from comment #32)
> I added the presentation that goes with cdt-examples to the repo.

Wonderful, thank you! I think the slides are all we need: https://github.com/jonahgraham/cdt-examples/blob/cdt_8_6/DSF%20Tutorial.pptx