Bug 84424

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 / TestsAssignee: 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 Flags
Fix the bug
none
Fixed per Mark's comment none

Description Mark Rowe (bdash) 2012-04-19 22:44:16 PDT
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.
Comment 1 Ryosuke Niwa 2012-04-19 22:47:47 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.
Comment 2 Ryosuke Niwa 2012-04-19 23:38:25 PDT
Created attachment 138050 [details]
Fix the bug
Comment 3 Mark Rowe (bdash) 2012-04-19 23:52:34 PDT
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?
Comment 4 Ryosuke Niwa 2012-04-19 23:53:50 PDT
Sure.
Comment 5 Ryosuke Niwa 2012-04-19 23:56:55 PDT
Created attachment 138052 [details]
Fixed per Mark's comment
Comment 6 WebKit Review Bot 2012-04-20 00:42:45 PDT
Comment on attachment 138052 [details]
Fixed per Mark's comment

Clearing flags on attachment: 138052

Committed r114725: <http://trac.webkit.org/changeset/114725>
Comment 7 WebKit Review Bot 2012-04-20 00:42:49 PDT
All reviewed patches have been landed.  Closing bug.