Bug 18487 - backwards compatibility support for nl/fr_FR_EURO
Summary: backwards compatibility support for nl/fr_FR_EURO
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Resources (show other bugs)
Version: 2.0   Edit
Hardware: PC All
: P1 normal (vote)
Target Milestone: 2.0 F3   Edit
Assignee: DJ Houghton CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-05-31 14:06 EDT by Jim des Rivieres CLA
Modified: 2002-06-06 17:42 EDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.