Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337748 - Source bundles are missing for Traditional Memory Rendering and p2
Summary: Source bundles are missing for Traditional Memory Rendering and p2
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-releng (show other bugs)
Version: 7.0   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 8.0.1   Edit
Assignee: Doug Schaefer CLA
QA Contact: Vivian Kong CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 271208
  Show dependency tree
 
Reported: 2011-02-21 13:06 EST by Martin Oberhuber CLA
Modified: 2011-09-13 12:23 EDT (History)
4 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 Martin Oberhuber CLA 2011-02-21 13:06:34 EST 
In CDT 7.0, source bundles are missing for following plugins:

   org.eclipse.cdt.debug.ui.memory.memorybrowser_1.2.1.201011171434.jar
   org.eclipse.cdt.debug.ui.memory.search_1.2.0.201011171434.jar
   org.eclipse.cdt.debug.ui.memory.traditional_1.2.0.201011171434.jar
   org.eclipse.cdt.debug.ui.memory.transport_2.1.0.201011171434.jar
   org.eclipse.cdt.p2_1.0.0.201011171434.jar

This is very problematic for adoption of CDT into our commercial product, since we want to ship source for all EPL code in order to comply with the EPL requirement of making source available -- this is easiest when source is shipped along with the binaries.

Could this please be fixed in a 7.0.2+ maintenance build such that we can adopt it into our product on top of Helios SR2.
Comment 1 Martin Oberhuber CLA 2011-03-11 16:33:03 EST 
Ping?

Not having source for all EPL bundles in CDT is a real problem for us.
We'd really appreciate getting this fixed!
Comment 2 Doug Schaefer CLA 2011-03-14 10:18:36 EDT 
cvs -d:pserver:anonymous@dev.eclipse.org:/cvsroot/tools co -r CDT_7_0_2 org.eclipse.cdt

That would be the easiest way to get CDT source for CDT 7.0.2 for example.
Comment 3 Martin Oberhuber CLA 2011-05-27 06:33:08 EDT 
Ping ?

Having source bundles not only aids in debugging, but also helps adopters
integrate TCF into their products. Since by shipping the source with the
product, the EPL requirement of "making source available" is most easily met.
Comment 4 Martin Oberhuber CLA 2011-06-28 04:49:25 EDT 
With cdt's releng being changed for git, could this issue please be addressed?

For our current product release, we've (again) run at risk not providing proper source due to a version mismatch. Getting source out of CVS is not a good option since the label cannot be computed automatically so it remains a manual step.

CDT does ship source bundles for all its other bundles. 
Please fix the releng process to provide source for these bundles as well.

Thanks!
Comment 5 Martin Oberhuber CLA 2011-09-07 07:37:01 EDT 
Ping, could this be considered for 8.0.1 ?
I hear that with Tycho, source bundle generation is easy...
Comment 6 Doug Schaefer CLA 2011-09-12 14:35:46 EDT 
I've added them. There is a new feature for memory.source and I have included the p2.source bundle with the p2 feature since it's small and the p2 feature shouldn't really be used by anyone.
Comment 7 CDT Genie CLA 2011-09-12 15:23:07 EDT 
*** cdt git genie on behalf of Doug Schaefer ***

    Bug 337748 - Added source for memory and p2.

[*] http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=45d46ea9791a8420d6ae6a0a5f03394c7151cbe8
Comment 8 CDT Genie CLA 2011-09-13 12:23:02 EDT 
*** cdt git genie on behalf of Doug Schaefer ***

    Bug 337748 - Added source for memory and p2.

[*] http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=eb8190211d7a181ba546d0725a6ddc90b7296be9