Issue 66669 - W32-bash patches
Summary: W32-bash patches
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: solenv (show other issues)
Version: current
Hardware: All Windows, all
: P3 Trivial (vote)
Target Milestone: ---
Assignee: quetschke
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-23 04:10 UTC by quetschke
Modified: 2013-08-07 15:34 UTC (History)
1 user (show)

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

Attachments
Patch for solenv/inc/wntmsci10.mk (587 bytes, patch)
2006-06-23 04:13 UTC, quetschke 		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 quetschke 2006-06-23 04:10:30 UTC 
Issue to collect W32-bash problems.
Comment 1 quetschke 2006-06-23 04:13:13 UTC 
Created attachment 37281 [details]
Patch for solenv/inc/wntmsci10.mk
Comment 2 quetschke 2006-06-23 15:23:34 UTC 
The previous patch should use $(PIPEERROR) instead of removing the

 |& $(GREP) -v "LNK4197:"

part. There are a *LOT* of these link warnings.

FYI: <http://go-oo.org/tinderbox/ause058/status.html>
Comment 3 hjs 2006-06-23 17:39:47 UTC 
any reason to remove completely (beside speed ;))
Comment 4 quetschke 2006-06-23 19:37:52 UTC 
No, the grep should stay. I was just curious to see how often this warning occurs.
Very often!!
Comment 5 quetschke 2006-06-24 03:03:58 UTC 
I committed the &| -> $(PIPEERROR) change to ause058.
Comment 6 hjs 2006-06-30 09:54:00 UTC 
done
Comment 7 hjs 2006-06-30 09:54:46 UTC 
please verify
Comment 8 quetschke 2006-07-03 00:24:33 UTC 
Works!
Comment 9 quetschke 2006-11-01 18:19:38 UTC 
close