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

Bug 367942

Summary:

setPDEBuildPath does nothing currently

Product:

[Tools] Linux Tools

Reporter:

Krzysztof Daniel <krzysztof.daniel>

Component:

eclipse-build

Assignee:

Packaging Tools Inbox <linux.packagingtools-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

akurtakov, krzysztof.daniel

Version:

unspecified

Target Milestone:

---

Hardware:

OS:

Linux

Whiteboard:

Attachments:

Description	Flags
First version of the patch	none
Fix in line with Alex suggestions	akurtakov: iplog+

Description Krzysztof Daniel

2012-01-05 08:00:41 EST

I think it is a side effect of fix for bug 366698 and earlier patches.

It looks like the setPDEBuildPath was designed to use PDE of Eclipse that was build. Maybe it is time to use PDEBuild from bootstrap?

Comment 1 Krzysztof Daniel

2012-01-05 08:01:54 EST

I forgot to mention setPDEBuildPath does nothing because it cannot discover PdeBuild in the build/<label>/installation folder, and therefore tests cannot be built.

Comment 2 Alexander Kurtakov

2012-01-05 08:14:17 EST

I have some concerns about using the bootstrap pde.build for running tests - it should be kept to the minimum and used only for building the sdk. The more we use it the harder to modify it it will become.
The correct fix for me seems to fix setPDEBuildPath to find pde.build in dropins which is the default now.

Comment 3 Krzysztof Daniel

2012-01-05 08:21:01 EST

Created attachment 209071 [details]
First version of the patch

Eclipse is built correctly. 
I am not sure about tests - build ends with success.

Comment 4 Krzysztof Daniel

2012-01-05 09:51:31 EST

Created attachment 209074 [details]
Fix in line with Alex suggestions

Comment 5 Alexander Kurtakov

2012-01-05 10:54:31 EST

Thanks.
Applied.