Bugzilla – Full Text Bug Listing
| Summary: | Host not found error in XL C/C++ compiler property page | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Tools] PTP | Reporter: | Vivian Kong <vivkong> | ||||||
| Component: | Remote Tools | Assignee: | Vivian Kong <vivkong> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||
| Severity: | normal | ||||||||
| Priority: | P3 | CC: | g.watson, john_ws_liu, jwsliu, recoskie | ||||||
| Version: | 4.0.6 | Flags: | recoskie:
review+
|
||||||
| Target Milestone: | 5.0 | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
Created attachment 195438 [details]
proposed patch for HEAD
Greg, Chris, John, can you please review this patch when you have a moment? The XLC compile property page is unusable for a remote tools connection without this fix. This patch introduces an API breakage but I think it is the right thing to do (since we shouldn't assume a remote project is using a RSE connection) and I don't think anyone is using this API. Also changed the name of the property page from "XL C/C++ Compiler" to "Remote XL C/C++ Compiler". I appreciate your feedback. Thanks. (In reply to comment #2) > Also changed the name of the property page from "XL C/C++ Compiler" to "Remote > XL C/C++ Compiler". That actually shouldn't be changed. Please ignore this change in the patch. I didn't try it, but based on looking at the code it seems ok to me. I applied the patch and did a brief test from my workspace, I didn't find any regression. Thanks for the reviews. Fixed in HEAD. |
Created attachment 184892 [details] screenshot Please see attached screenshot for the error. RemoteXLCompilerPropertyPage uses the project location to figure out the remote host and create a path editor for the compiler path. However it assumes the project location is a RSE location.