Summary: | REGRESSION(r112781): Windows build slaves are no longer including .pdb files in archives handed to testers | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Rowe (bdash) <mrowe> | ||||||
Component: | Tools / Tests | Assignee: | Ryosuke Niwa <rniwa> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | rniwa, tony, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Mark Rowe (bdash)
2012-04-19 22:44:16 PDT
(In reply to comment #0) > I noticed while poking around this evening that the Windows build slaves are no longer including .pdb files in the .zip file that they upload to the master for the test slaves to use. The change in question appears to have been made for the benefit of Chromium bots, so it's unclear why these files are being excluded from non-Chromium builds. Oh, I didn't know that. Let me upload a patch then. Created attachment 138050 [details]
Fix the bug
Comment on attachment 138050 [details] Fix the bug I'm not sure how I feel about this change. I'm not sure how much sense it makes for the patterns to be shared like this given that there's very little overlap between the patterns used by Windows vs Chromium. The Windows code only wants to exclude *.ilk (that's what it did prior to r112781). Can the callers pass in the array of patterns instead, with the Windows and Chromium sets being defined as module-level variables? Sure. Created attachment 138052 [details]
Fixed per Mark's comment
Comment on attachment 138052 [details] Fixed per Mark's comment Clearing flags on attachment: 138052 Committed r114725: <http://trac.webkit.org/changeset/114725> All reviewed patches have been landed. Closing bug. |