Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312509 - EDCSymbolReader.java build error.
Summary: EDCSymbolReader.java build error.
Status: RESOLVED INVALID
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-edc (show other bugs)
Version: 0 DD 1.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Ken Ryall CLA
QA Contact: Ken Ryall CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-11 16:36 EDT by Dave Korn CLA
Modified: 2010-05-28 11:55 EDT (History)
1 user (show)

See Also:

Attachments
Simple fix. (1.17 KB, patch)
2010-05-11 16:39 EDT, Dave Korn CLA 		no flags Details | Diff
Show Obsolete (1) View All 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 Dave Korn CLA 2010-05-11 16:36:26 EDT 
Build Identifier: I20100429-1549

With current CVS HEAD, I get the following error during compilation of org.eclipse.cdt.debug.edc.internal.symbols.dwarf/EDCSymbolReader.java

The type EDCSymbolReader must implement the inherited abstract method ISymbolReader.getSourceFiles(IProgressMonitor)



Reproducible: Always

Steps to Reproduce:
1. Install M7 SDK + (fixed) cdt master M7 release.
2. Check out cdt tools from dev.eclipse.org CVS
3. Build all.
Comment 1 Dave Korn CLA 2010-05-11 16:39:07 EDT 
Created attachment 168032 [details]
Simple fix.

I trivially copypasta'd the definition of the missing function from the other symbol readers that do implement it:
org.eclipse.cdt.utils.{coff/CodeViewReader,debug.dwarf/DwarfReader,debug.stabs/StabsReader}.java

It fixes the build for me.
Comment 2 John Cortell CLA 2010-05-11 16:40:35 EDT 
EDCSymbolReader does implement that method. It was added last Friday. Make sure you have the latest (rev 1.5) of EDCSymbolReader. Sounds to me you're not completely in synch with the repository.
Comment 3 Dave Korn CLA 2010-05-11 16:46:09 EDT 
(In reply to comment #2)
> EDCSymbolReader does implement that method. It was added last Friday. Make sure
> you have the latest (rev 1.5) of EDCSymbolReader. Sounds to me you're not
> completely in synch with the repository.

How bizarre.  I updated using the team synch view last night, but for some reason I still only have 1.2; yet I got a whole lot of other updates.  Sorry for the false alarm.  (I may turn out to have found a bug in the team sync view, or maybe I've mangled my cvs metadata somehow, I'll open a fresh bug if it turns out to be the former.)  Sorry for the noise.