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

Bug 369204

Summary: Hardcoded dependency to IBM Java in or.eclipse.core.filesystem
Product: [Eclipse Project] Platform Reporter: Krzysztof Daniel <krzysztof.daniel>
Component: ResourcesAssignee: Krzysztof Daniel <krzysztof.daniel>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: krzysztof.daniel, remy.suen, sptaszkiewicz
Version: 3.8   
Target Milestone: 3.8 M6   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Fix proposition
none
Patch without additional changes none

Description Krzysztof Daniel CLA 2012-01-20 06:46:46 EST
Created attachment 209805 [details]
Fix proposition

The plugin should not be dependent on any vendor. Ideally environmental JAVA_HOME should be used.

Additionally, more optimization could be used.
Comment 1 John Arthorne CLA 2012-01-20 08:54:13 EST
Can you explain your changes to the OPT_FLAGS?
Comment 2 Krzysztof Daniel CLA 2012-01-20 11:16:07 EST
Those changes are not substantial to the patch:

(1)replacement of -O with -O2
   this improves the level of code optimization
(2) -g adds gdb debugging symbols.
Comment 3 Krzysztof Daniel CLA 2012-03-05 05:37:39 EST
Created attachment 212056 [details]
Patch without additional changes
Comment 4 Krzysztof Daniel CLA 2012-03-05 05:55:13 EST
The remaining changes from the initial patch have now their own bug 373232.
Comment 5 Szymon Brandys CLA 2012-03-05 07:20:06 EST
Krzysztof, can you also build natives?
Comment 6 John Arthorne CLA 2012-03-05 10:46:16 EST
This patch doesn't affect the compiled natives so we don't need to produce new ones here. The patch just means the person calling make has to configure JAVA_HOME rather than having it configured in the makefile itself.
Comment 7 Szymon Brandys CLA 2012-03-05 16:36:09 EST
Krzysztof, please submit the latest patch via Gerrit. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=371767. 

Krzysztof, my question about building natives is still valid. Just the part that requires new natives is moved to bug 373232. So let's talk there.
Comment 8 Krzysztof Daniel CLA 2012-03-06 03:05:47 EST
Patch in gerrit: https://git.eclipse.org/r/5258
Comment 9 Krzysztof Daniel CLA 2012-03-08 08:45:00 EST
The patch has been released.
Comment 10 Krzysztof Daniel CLA 2012-03-08 09:32:12 EST
I forgot to mark it as resolved.