Issue 93435 - mergemodule.pm does not properly normalize paths for cygwin
Summary: mergemodule.pm does not properly normalize paths for cygwin
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: solenv (show other issues)
Version: current
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.0
Assignee: ingo.schmidt-rosbiegal
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-03 16:38 UTC by thb
Modified: 2008-09-16 16:59 UTC (History)
5 users (show)

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

Attachments
Changes as in CVS (1.84 KB, patch)
2008-09-04 16:50 UTC, thb 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description thb 2008-09-03 16:38:01 UTC 
See $subject. All paths need to go through cygpath. Fixing in CWS kendy21.
Comment 1 thb 2008-09-03 16:39:32 UTC 
Fixed in CWS kendy21.
Comment 2 thb 2008-09-03 16:41:02 UTC 
@is: please verify in CWS (I think a quick check of _that_ change in a 4NT env
might be in order, just to be on the safe side of things).
Comment 3 thb 2008-09-03 16:52:49 UTC 
gnah. part of this is already fixed with issue 93065 as I now realize. But
there's more to it (and fixed herewith), will resync this one file in a minute.
Comment 4 thb 2008-09-04 16:50:04 UTC 
Created attachment 56245 [details]
Changes as in CVS
Comment 5 thb 2008-09-04 16:52:45 UTC 
@is: added a patch with the relevant changes for your convenience. Could you (or
someone else at Sun) please quickly verify that this does not break 4NT build
(just apply to m31, and see whether instset_native still works)? The surrounding
CWS is otherwise ready for approval.
Comment 6 ingo.schmidt-rosbiegal 2008-09-05 10:33:31 UTC 
@thb: I think, that this is even relevant for OOo 3.0 -> changing target. I will
try to integrate this in my cws native196.

@mba: Can you please check this for dev300 builds?
Comment 7 ingo.schmidt-rosbiegal 2008-09-05 11:13:07 UTC 
IS: Patch does not influence 4NT builds -> verified.
Comment 8 ingo.schmidt-rosbiegal 2008-09-16 16:59:44 UTC 
integrated into ooo300 m6 -> closing issue.