Issue 78061 - Normalize multiple leading slashes correctly
Summary: Normalize multiple leading slashes correctly
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: dmake (show other issues)
Version: current
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: hjs
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-03 17:22 UTC by quetschke
Modified: 2013-08-07 15:34 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description quetschke 2007-06-03 17:22:47 UTC
Teach dmake to normalize leading slashes according to POSIX rules. (One or more
than two leading slashes -> /  and keep // as it has a special meaning.)

Also reduce multiple slashes after a dos drive letter to exactly one slash.
(c:\\\\bla -> c:/blah)

See issue 74007 for the patch.
Comment 1 quetschke 2007-06-22 21:47:42 UTC
Committed testcase and documentation.
Comment 2 quetschke 2007-06-22 21:49:18 UTC
Please verify.
Comment 3 hjs 2007-06-26 18:34:58 UTC
peeking around:

found a rather pathological case which doesn't normalize well to posix notation:
c:/ff/./\\pp -> /cygdrive/c/ff//pp

it looks like backslashes are only handled on windows, ignored elsewhere. 

Comment 4 quetschke 2007-06-26 19:12:17 UTC
Oeh! Well, this is actually an oversight from issue 69742. The patch from
this issue only touched leading /s or the first slashes behind a drive letter.

Nice catch ;) but not relevant, I hope. Can I fix this in dmake411 ?
Comment 5 hjs 2007-06-28 15:47:12 UTC
.
Comment 6 hjs 2007-07-13 14:30:04 UTC
dmake 4.9 active in MWS