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

Bug 336257

Summary:

[dstore] leading file.searator in DSTORE_LOG_DIRECTORY not handled

Product:

[Tools] Target Management

Reporter:

Onno Van den Troost <onno>

Component:

RSE

Assignee:

David McKnight <dmcknigh>

Status:

RESOLVED FIXED

QA Contact:

Martin Oberhuber <mober.at+eclipse>

Severity:

normal

Priority:

CC:

dmcknigh

Version:

3.0.3

Target Milestone:

3.3 M6

Hardware:

Other

OS:

other

Whiteboard:

Bug Depends on:

Bug Blocks:

336350, 336351

Attachments:

Description	Flags
patch to remote leading / from log directory	none

Description Onno Van den Troost

2011-02-03 11:36:01 EST

getUserPreferencesDirectory() checks for the DSTORE_LOG_DIRECTORY property to allow changing the preferences directory.
In this process, getUserPreferencesDirectory() ensures there's a trailing file.separator, but it does not check for a leading one. 
This can result in a double file.separator in the path as DSTORE_LOG_DIRECTORY is appended to _userPreferencesDirectory which has a trailing file.separator.

Please backport to 3.0.3 and 3.2.1

Comment 1 David McKnight

2011-02-04 09:44:47 EST

Created attachment 188321 [details]
patch to remote leading / from log directory

Comment 2 David McKnight

2011-02-04 09:55:38 EST

I've committed the fix to cvs.

I've opened bug 336350 for the 3.2.x backport and I've opend bug 336351 for the 3.0.x backport.