Summary: | Unable to build source files in a linked directory | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Tools] CDT | Reporter: | Hurai Rody <hurai_r> | ||||||
Component: | cdt-build | Assignee: | Sean Evoy <sevoy> | ||||||
Status: | VERIFIED FIXED | QA Contact: | |||||||
Severity: | major | ||||||||
Priority: | P3 | CC: | cdtdoug, recoskie | ||||||
Version: | 2.0 | ||||||||
Target Milestone: | 2.0.1 | ||||||||
Hardware: | All | ||||||||
OS: | Windows XP | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Hurai Rody
2004-07-20 17:34:03 EDT
Will test the fix and if it works I will add it to the default makefile generator Created attachment 13924 [details]
proposed patch
This patch is largely the same as the one posted by the submitter, however, the
submitter's patch did not take into account the relative path of the object
file in the project. Currently there is a restriction (for unknown reasons)
that linked files can only be in the project root directory, but I am hoping
that this restriction will disappear, and hence it would be good to allow for
this case.
Applied the patch to 2.0.1 and head. The switch to absolute paths is probably the only way to go until we have a strategy in place to define and use path variables. There are two usability probalems that will not be addressed with thispatch. Despite the fact that the resource "looks" like it is part of the worksapce, the user will have to set the proper includes path in the build settings or the compielr will failt o find it. Second, error markers are not cleared if the linked resource belongs to another, open project in the workspace. Created attachment 14445 [details]
NPE in generated makefile builder
These log entries are added when I build a project containing a linked
directory to a mixed source/object directory.
Found out that the NPEs would also happen if the object files were in a non-linked directory as well, so now we can mark this fix for linked dir as verified. I verified in CDT 2.0.1 build 200408271356. Marking it verified as per Tany's comment |