Summary: | svn-apply Perl error seen on commit-queue | ||
---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WORKSFORME | ||
Severity: | Normal | CC: | joepeck |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | PC | ||
OS: | OS X 10.5 |
Description
Eric Seidel (no email)
2009-11-09 17:20:16 PST
VCSUtils.pm has been edited multiple times since 2009-11-09. Cameron Zwarich gave me great advice the other day when I posted a bug on an ASSERT crash. He said you should also include the source line, in case files changes. That can save a lot of digging in the future. I did the digging, and it looks like line 397 corresponded to line 396 in r50547, inside the fixChangeLogPatch function: http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/VCSUtils.pm?rev=50547#L363 That has since moved to line 694!, has undergone some changes, and now includes unit tests! http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/VCSUtils.pm#L672 Have you seen this problem crop up again recently or can this bug be closed? Yes, things are way better than when I filed this bug. :) |