Bug 18487

Summary: backwards compatibility support for nl/fr_FR_EURO
Product: [Eclipse Project] Platform Reporter: Jim des Rivieres <jeem>
Component: ResourcesAssignee: DJ Houghton <dj.houghton>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P1 CC: dean.t.roberts, debbie_wilson, John_Wiegand
Version: 2.0   
Target Milestone: 2.0 F3   
Hardware: PC   
OS: All   
Whiteboard:

Description Jim des Rivieres CLA 2002-05-31 14:06:50 EDT
In Eclipse 1.0, Core runtime's "$nl$" mechanism was implemented
incorrectly.  A path like "$nl$/foo.txt" would end up looking in
the following locale-specific locations under the plug-in (including
fragments), assuming the current locale is "fr_FR_EURO":
    nl/fr_FR_EURO/foo.txt
    nl/fr_FR/foo.txt
    nl/fr/foo.txt
    foo.text
This is incorrect. It should have been looking in this sequence
of locations:
    nl/fr/FR/EURO/foo.txt
    nl/fr/FR/foo.txt
    nl/fr/foo.txt
    foo.text

This problem has been corrected For Eclipse 2.0

However, for backwards compatibility, the old 1.0 way should be supported
as well (considered as deprecated). The implementation should look in this 
order:
    nl/fr/FR/EURO/foo.txt
    nl/fr_FR_EURO/foo.txt
    nl/fr/FR/foo.txt
    nl/fr_FR/foo.txt
    nl/fr/foo.txt
    foo.text
Comment 1 DJ Houghton CLA 2002-06-01 00:35:38 EDT
Consider for FP2.
Comment 2 DJ Houghton CLA 2002-06-06 09:45:24 EDT
Requires changes in:
  PluginDescriptor.buildVariants
  PluginDescriptor.findNL

Comment 3 DJ Houghton CLA 2002-06-06 17:42:21 EDT
Fixed. Added test cases. Reviewed. Released.
Closing.