Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 344694 - Send full path of source file to gdb while inserting breakpoint in dsf
Summary: Send full path of source file to gdb while inserting breakpoint in dsf
Status: NEW
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf-gdb (show other bugs)
Version: 8.0   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact: Jonah Graham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-04 08:13 EDT by Abeer Bagul CLA
Modified: 2020-09-04 15:19 EDT (History)
2 users (show)

See Also:

Attachments
Patch to allow sub-service of MIBreakpointsManager to override adjustDebuggerPath (1.96 KB, patch)
2011-05-04 08:14 EDT, Abeer Bagul CLA 		no flags Details | Diff
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 Abeer Bagul CLA 2011-05-04 08:13:12 EDT 
Build Identifier: I20110310-1119

The method MIBreakpointsManager.adjustDebuggerPath(...) has been added to fix
DSF bug 232415.
This DSF bug reports that mingw-gdb has problems accepting windows style paths
while inserting breakpoints, hence the method truncates the path to only
filename. This has the downside of not being able to resolve duplicate
filenames.

Our gdb is able to accept windows style paths, hence it should be possible to override this method in a subclass of MIBreakpointsManager and make it a noop.


Reproducible: Always
Comment 1 Abeer Bagul CLA 2011-05-04 08:14:36 EDT 
Created attachment 194702 [details]
Patch to allow sub-service of MIBreakpointsManager to override adjustDebuggerPath
Comment 2 Marc Khouzam CLA 2011-05-04 09:15:09 EDT 
Do you need this for this release?  We are past API freeze, so I would prefer to wait until the next release.  Is that ok for you?
Comment 3 Abeer Bagul CLA 2011-05-05 02:35:11 EDT 
Sure, we already have added the same patch in our copy of CDT, so this can wait.
Comment 4 Marc Khouzam CLA 2011-05-05 10:34:06 EDT 
(In reply to comment #3)
> Sure, we already have added the same patch in our copy of CDT, so this can
> wait.

Thanks.  You'll probably have to remind me to get back to this once we have the branch for the next release.