Bug 83342 - webkit-patch upload --suggest-reviewers shouldn't break in the presence of deleted or moved files
Summary: webkit-patch upload --suggest-reviewers shouldn't break in the presence of de...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-05 18:27 PDT by Ami Fischman
Modified: 2016-07-28 13:22 PDT (History)
8 users (show)

See Also:


Attachments
Patch v1 (6.54 KB, patch)
2016-07-28 12:47 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ami Fischman 2012-04-05 18:27:33 PDT
Apparently webkit-patch upload's --suggest-reviewers feature is broken in the presence of moved (removed?) files.  After moving a test & its expectation from LayoutTests/http/tests/media to LayoutTests/media the upload failed like this:

$ ./Tools/Scripts/webkit-patch upload --request-commit --suggest-reviewers 83296
Total errors found: 0 in 4 files
Was that diff correct? [Y/n]:
Fetching: https://bugs.webkit.org/show_bug.cgi?id=83296&ctype=xml
Path /usr/local/google/fischman/src/chromium/src/third_party/WebKit/LayoutTests/http/tests/media/media-can-load-when-hidden-expected.txt does not exist.
Path /usr/local/google/fischman/src/chromium/src/third_party/WebKit/LayoutTests/http/tests/media/media-can-load-when-hidden-expected.txt does not exist.

Removing the --suggest-reviewers option made it work just fine.
Comment 1 David Kilzer (:ddkilzer) 2016-07-28 12:47:46 PDT
Created attachment 284804 [details]
Patch v1
Comment 2 WebKit Commit Bot 2016-07-28 12:50:37 PDT
Attachment 284804 [details] did not pass style-queue:


ERROR: Tools/Scripts/webkitpy/common/checkout/scm/scm_unittest.py:413:  [SCMTest._shared_test_revisions_changing_file] Instance of 'SCMTest' has no 'scm' member  [pylint/E1101] [5]
Total errors found: 1 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 David Kilzer (:ddkilzer) 2016-07-28 12:52:38 PDT
(In reply to comment #2)
> Attachment 284804 [details] did not pass style-queue:
> 
> ERROR: Tools/Scripts/webkitpy/common/checkout/scm/scm_unittest.py:413: 
> [SCMTest._shared_test_revisions_changing_file] Instance of 'SCMTest' has no
> 'scm' member  [pylint/E1101] [5]
> Total errors found: 1 in 4 files

This can be safely ignored.  Tests pass despite this warning.
Comment 4 WebKit Commit Bot 2016-07-28 13:22:15 PDT
Comment on attachment 284804 [details]
Patch v1

Clearing flags on attachment: 284804

Committed r203832: <http://trac.webkit.org/changeset/203832>
Comment 5 WebKit Commit Bot 2016-07-28 13:22:20 PDT
All reviewed patches have been landed.  Closing bug.