Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 315788

Summary: need workaround for the fact that scanner info paths are returned in client's path format
Product: [Tools] PTP Reporter: Chris Recoskie <recoskie>
Component: RDTAssignee: Chris Recoskie <recoskie>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 CC: g.watson, vivkong
Version: 4.0Flags: vivkong: review+
g.watson: review+
Target Milestone: 4.0   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
proposed patch recoskie: iplog-

Description Chris Recoskie CLA 2010-06-04 11:33:38 EDT 
Unfortunately we are victim of a CDT problem (Bug 315632), where the include paths returned from the project description provider are all returned using the path format of the local machine.

This really screws things up when your local machine is windows and the remote machine is *NIX.  The preprocessor on the remote system gets some paths containing backslashes, and since backslash is a valid filename character, it treats it as such and looks for a big long filename full of backslashes, which of course it doesn't find.

It's not a real solution, but a temporary hack-fix is to convert all backslashes to slashes before sending the paths to the remote system.  This way at least the main RDT use cases of Windows->*NIX and *NIX->*NIX will work.

Unfortunately, using Windows as a remote system will be busted until we have a real fix for this.
Comment 1 Chris Recoskie CLA 2010-06-04 11:35:10 EDT 
Created attachment 171112 [details]
proposed patch

Attached proposed patch.
Comment 2 Greg Watson CLA 2010-06-04 11:48:37 EDT 
I agree it's a hack, but it seems ok until someone wants to use Windows as a backend. We should specify which backend systems are supported somewhere if we don't already.
Comment 3 Chris Recoskie CLA 2010-06-04 11:52:38 EDT 
BTW this also means that using backslshes in your filenames won't work either.  Thankfully, most people have the sense not to do that though.
Comment 4 Chris Recoskie CLA 2010-06-08 14:05:34 EDT 
This was committed to HEAD.