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

Bug 326613

Summary:

test credentials lookup does not work when workspace path contains spaces

Product:

z_Archived

Reporter:

Thomas Ehrnhoefer <thomas.ehrnhoefer>

Component:

Mylyn

Assignee:

Thomas Ehrnhoefer <thomas.ehrnhoefer>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

steffen.pingel, thomas.ehrnhoefer

Version:

unspecified

Keywords:

contributed

Target Milestone:

3.4.3

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
patch	steffen.pingel: iplog+, thomas.ehrnhoefer: review?
mylyn/context/zip	none
patch v2	steffen.pingel: iplog+, thomas.ehrnhoefer: review?
mylyn/context/zip	none

Description Thomas Ehrnhoefer

2010-09-29 19:26:44 EDT

The file separator replacement fails, as well as encoding problems of the path prevent headless tests from using the testUtil's credentials.

Comment 1 Thomas Ehrnhoefer

2010-09-29 19:28:36 EDT

Created attachment 179910 [details]
patch

this fixes the problem for me (win 7)
Steffen, could you try if it still works on linux?

Comment 2 Thomas Ehrnhoefer

2010-09-29 19:28:37 EDT

Created attachment 179911 [details]
mylyn/context/zip

Comment 3 Steffen Pingel

2010-09-29 23:02:41 EDT

Thanks Thomas. Patch applied to e_3_6_m_3_x branch and head.

Comment 4 Thomas Ehrnhoefer

2010-10-04 12:55:20 EDT

Steffen, your change to the patch breaks this on windows again. The problem is that the replaceAll method runs into trouble if the replacement string is a "\". Running the decoder earlier wont fix that.

Comment 5 Thomas Ehrnhoefer

2010-10-04 12:55:53 EDT

Created attachment 180183 [details]
patch v2

Comment 6 Thomas Ehrnhoefer

2010-10-04 12:55:55 EDT

Created attachment 180184 [details]
mylyn/context/zip

Comment 7 Steffen Pingel

2010-10-05 02:19:32 EDT

My bad. Thanks for catching that.  I have applied the patch to the e_3_6_m_3_x branch and head.