Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 464191 - "org.eclipse.rcp.configuration_root.cocoa.macosx.x86(_64)" p2 IU does not delete eclipse symlink during uninstall
Summary: "org.eclipse.rcp.configuration_root.cocoa.macosx.x86(_64)" p2 IU does not del...
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 4.2   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: 4.2.2+   Edit
Assignee: David Williams CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 407775
Blocks:
  Show dependency tree
 
Reported: 2015-04-08 13:52 EDT by David Williams CLA
Modified: 2015-04-11 01:20 EDT (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2015-04-08 13:52:01 EDT
+++ This bug was initially created as a clone of Bug #407775 +++

cloning original, just to have a clear target for 4.2.2+ work.
Comment 1 David Williams CLA 2015-04-08 13:57:14 EDT
I've committed similar fix as in 4.4 and 4.5 (though, in 4.5, later removed, as we no longer have sym links in 4.5, with new mac app layout). 


http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?h=R4_2_maintenance&id=bb72a6e21f8921f58eb86da54e82955615ce97fe
Comment 2 David Williams CLA 2015-04-08 14:00:40 EDT
Meant to add ... 

Markus, if you get a chance to proof read, wouldn't hurt. (given the "typos" I made the first time!) 

Steve, if you can do an LTS build of 422+ some time (and remind me where to get the results :) I can do a quick sanity check that the "uninstall" script in original build does remove the sym link as intended. 

This should only effect macosx products.
Comment 3 David Williams CLA 2015-04-08 14:15:22 EDT
Just realized, given the title of the bug, that this fix should to in "configuration feature" too, not just our "products". 

I copied the tooling IUs from "platform product" to "configuration feature". 

http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?h=R4_2_maintenance&id=dac964d2884f48ac772f07d14a730f51e7088afe

Given they have the same name, only one would "apply" to a given install, but, should matter which, since they are the same. 

Comments welcome.
Comment 4 David Williams CLA 2015-04-09 21:17:07 EDT
I've reverted the changes I've made here. 

I forgot where I was. :/ 

Things "way back then" are a little more confusing, and my initial fix end up with no symbolic link being creating, just copying the executable to the root directory (in short, because some things were "done twice" back then), so "order" is critical, and unpredictable). 

I'll need to give this some more thought, for the 422 stream to determine if there is a way to fix it, without a "complete cleanup" as we've done in subsequent streams.
Comment 5 David Williams CLA 2015-04-11 01:20:43 EDT
After some IMing with some interested parties, it was decided not bother with this fix. 

There is no easy way to deliver it, and they can work around it (with ant tasks, I believe) so "cost" doesn't cover "benefit".