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

Bug 99704

Summary: Eclipse application launch configuration file format should be platform independent
Product: [Eclipse Project] PDE Reporter: Michal Tkacz <Michal.Tkacz>
Component: UIAssignee: PDE-UI-Inbox <pde-ui-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 CC: andre_weinand
Version: 3.1   
Target Milestone: 3.2 M2   
Hardware: PC   
OS: All   
Whiteboard:

Description Michal Tkacz CLA 2005-06-13 11:50:09 EDT 
Right now launch configuration for an Eclipse application, when stored in a
file, uses platform dependent path separator to separate plugins to be launched.
This makes it impossible to share the configuration between different platforms.
This problem may affect other types of launch configurations as well.
Comment 1 Wassim Melhem CLA 2005-06-13 19:21:59 EDT 

*** This bug has been marked as a duplicate of 86576 ***
Comment 2 Wassim Melhem CLA 2005-06-13 19:22:24 EDT 
sorry, wrong dup.
Comment 3 Wassim Melhem CLA 2005-06-13 19:24:16 EDT 

*** This bug has been marked as a duplicate of 86796 ***
Comment 4 Michal Tkacz CLA 2005-06-14 03:26:36 EDT 
I really don't think this is a duplicate of bug 86796. Bug 86796 is about string
substitution variables, while this bug is not affected by these variables in any
way. Note also that the workspace location parameter works exactly as expected
by the user i.e. it is saved and restored from the file correctly on different
platforms, so bug 86796 is simly an enhancement. On the other hand the list of
plugins to be launched is not restored correctly on a different platform
(resulting in an empty selection), because it assumes that both platforms use
the same path separator.

Although both bugs aim to make the launch configuration more portable, I think
this bug could be addressed separately by replacing File.pathSeparatorChar with
a fixed character during save and replacing that fixed character with
File.pathSeparatorChar during load. Please consider reopening it.
Comment 5 Wassim Melhem CLA 2005-06-14 08:12:35 EDT 
oh you're right. reopening.
Comment 6 Wassim Melhem CLA 2005-06-14 10:57:19 EDT 
*** Bug 99965 has been marked as a duplicate of this bug. ***
Comment 7 Andre Weinand CLA 2005-06-14 11:05:42 EDT 
Adjusting severity to major because it makes all our launch configs (that we were creating for cross 
platform tests) worthless.
Comment 8 Wassim Melhem CLA 2005-09-18 11:52:07 EDT 
Fixed.  The delimiter is now ','.  We will provide backward compatibility for 
all existing launch configs.
Comment 9 Andre Weinand CLA 2005-09-18 13:25:50 EDT 
Great! Thanks Wassim.
This ends a nightmare for all Mac developers!
(and I still believe the fix was trivial...)
Comment 10 Wassim Melhem CLA 2005-09-18 13:37:41 EDT 
the fix for this particular defect was indeed trivial.

However, to make your launch configurations 100% portable required more work 
(support for variable substitutions, etc.).  It will all be in 3.2M2