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

Bug 225042

Summary: build.properties doesn't seem right
Product: z_Archived Reporter: David Williams <david_williams>
Component: WTP IncubatorAssignee: David Williams <david_williams>
Status: VERIFIED FIXED QA Contact: David Williams <david_williams>
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch to add src-validation folder, and remove others none

Description David Williams CLA 2008-04-01 01:05:27 EDT
Created attachment 94323 [details]
patch to add src-validation folder, and remove others

in xsl.core, that is. 

Most important, I think src-validation needs to be included in the source.. property. 

Also important, the src.includes property includes too much. As a general rule, only things NOT listed int the binary build need to be listed here ... such as "sdk documentation" and such. In fact, an oddity of PDE builds, including the source folders in the source properties causes it to be included twice (at least, used to, before jarred source files). 

I think the attached patch for build.properties would be worth a try ... though might need some tweaks.
Comment 1 David Williams CLA 2008-04-01 07:58:32 EDT
I see there's been 3 compile errors in xsl for a while, that I think are the result of the src-validation not being in build.properties, so I went ahead and released. 

Someone will still need to verify that the SDK version is correct (has schemas, etc., just like the binary version). 

Comment 2 David Carver CLA 2008-04-02 21:13:05 EDT
David can you make sure the prefix these with [xslt] if possible.   If it doesn't have that prefix I tend to miss these type of bugs as being general incubator bugs.