Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323755 - [tracepoints] NLS strings gone missing due to lowercase file name
Summary: [tracepoints] NLS strings gone missing due to lowercase file name
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf-gdb (show other bugs)
Version: 8.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 8.0   Edit
Assignee: Marc Khouzam CLA
QA Contact: Marc Khouzam CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-26 16:21 EDT by Marc Khouzam CLA
Modified: 2010-08-27 14:19 EDT (History)
2 users (show)

See Also:

Attachments
Fix (6.20 KB, patch)
2010-08-26 16:23 EDT, Marc Khouzam CLA 		marc.khouzam: iplog-
Details | Diff
View All 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 Marc Khouzam CLA 2010-08-26 16:21:39 EDT 
The file org.eclipse.cdt.dsf.gdb.internal.service.command.events.Messages was recently modified to use its class name to find the corresponding properties file.  The problem is that the properties file is named org.eclipse.cdt.dsf.gdb.internal.service.command.events.messages.properties, which is in lower case.  Therefore, on Linux, the properties file is not found.

The patch I will attach fixes the issue.  Note that I had to rename both classes.  The reason is that changing the case of a file causes problems in Windows when updating from CVS, because Windows is not case sensitive.  An actual name change is required for Windows to notice the different.  These classes are internal so changing their name is ok.
Comment 1 Marc Khouzam CLA 2010-08-26 16:23:44 EDT 
Created attachment 177562 [details]
Fix

The patch I committed to HEAD.
Comment 2 Marc Khouzam CLA 2010-08-26 16:25:47 EDT 
Sergey do you know if there were other such changes that might have caused a similar problem?
Comment 3 Sergey Prigogin CLA 2010-08-27 01:35:09 EDT 
(In reply to comment #2)
> Sergey do you know if there were other such changes that might have caused a
> similar problem?

I may have renamed few messages.properties files to Messages.properties, but it was couple months ago and nobody complained. The automatic builds could not be affected not affected since they pull everything from CVS every time.
Comment 4 Marc Khouzam CLA 2010-08-27 09:36:56 EDT 
(In reply to comment #3)
> (In reply to comment #2)
> > Sergey do you know if there were other such changes that might have caused a
> > similar problem?
> 
> I may have renamed few messages.properties files to Messages.properties, but it
> was couple months ago and nobody complained. The automatic builds could not be
> affected not affected since they pull everything from CVS every time.

messages.properties to Messages.properties is not such a big deal.
What I'm more curious about is if you removed other hard-coded bundle names in classes such as

private static final String BUNDLE_NAME = "org.eclipse.cdt.dsf.gdb.internal.service.command.events.messages"; //$NON-NLS-1$

to use Messages.class.getName() instead.  The case-sensitive aspect of this change could break other string displays.

I will check the commit mailing list for that date.  No biggy.
Comment 5 Marc Khouzam CLA 2010-08-27 14:19:46 EDT 
(In reply to comment #4)

> I will check the commit mailing list for that date.  No biggy.

I checked: looks good.