Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326613 - test credentials lookup does not work when workspace path contains spaces
Summary: test credentials lookup does not work when workspace path contains spaces
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.4.3   Edit
Assignee: Thomas Ehrnhoefer CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2010-09-29 19:26 EDT by Thomas Ehrnhoefer CLA
Modified: 2011-01-03 21:06 EST (History)
2 users (show)

See Also:

Attachments
patch (1.33 KB, patch)
2010-09-29 19:28 EDT, Thomas Ehrnhoefer CLA 		steffen.pingel: iplog+
thomas.ehrnhoefer: review?
Details | Diff
mylyn/context/zip (1.56 KB, application/octet-stream)
2010-09-29 19:28 EDT, Thomas Ehrnhoefer CLA 		no flags Details
patch v2 (1.18 KB, patch)
2010-10-04 12:55 EDT, Thomas Ehrnhoefer CLA 		steffen.pingel: iplog+
thomas.ehrnhoefer: review?
Details | Diff
mylyn/context/zip (1.03 KB, application/octet-stream)
2010-10-04 12:55 EDT, Thomas Ehrnhoefer CLA 		no flags Details
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 Thomas Ehrnhoefer CLA 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 CLA 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 CLA 2010-09-29 19:28:37 EDT 
Created attachment 179911 [details]
mylyn/context/zip
Comment 3 Steffen Pingel CLA 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 CLA 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 CLA 2010-10-04 12:55:53 EDT 
Created attachment 180183 [details]
patch v2
Comment 6 Thomas Ehrnhoefer CLA 2010-10-04 12:55:55 EDT 
Created attachment 180184 [details]
mylyn/context/zip
Comment 7 Steffen Pingel CLA 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.