Bug 163478 - REGRESSION (r206973): Running "webkit-patch suggest-reviewers" throws an AttributeError: 'NoneType' object has no attribute 'full_name'
Summary: REGRESSION (r206973): Running "webkit-patch suggest-reviewers" throws an Attr...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords:
Depends on: 163183
Blocks:
  Show dependency treegraph
 
Reported: 2016-10-14 18:23 PDT by Andy Estes
Modified: 2016-10-14 21:21 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.62 KB, patch)
2016-10-14 20:26 PDT, Simon Fraser (smfr)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Estes 2016-10-14 18:23:11 PDT
% webkit-patch suggest-reviewers
Traceback (most recent call last):
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkit-patch", line 84, in <module>
    main()
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkit-patch", line 79, in main
    WebKitPatch(os.path.abspath(__file__)).main()
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main
    result = command.check_arguments_and_execute(options, args, self)
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute
    self._sequence.run_and_handle_errors(tool, options, state)
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors
    self._run(tool, options, state)
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run
    step(tool, options).run(state)
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/tool/steps/suggestreviewers.py", line 45, in run
    reviewers = self._tool.checkout().suggested_reviewers(self._options.git_commit, self._changed_files(state))[:5]
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/common/checkout/checkout.py", line 146, in suggested_reviewers
    unique_reviewers = reduce(lambda suggestions, reviewer: suggestions + [reviewer if reviewer not in suggestions else None], reviewers, [])
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/common/checkout/checkout.py", line 146, in <lambda>
    unique_reviewers = reduce(lambda suggestions, reviewer: suggestions + [reviewer if reviewer not in suggestions else None], reviewers, [])
  File "/Users/andy/Code/OpenSource/Tools/Scripts/webkitpy/common/config/committers.py", line 79, in __eq__
    return (self.full_name == other.full_name
AttributeError: 'NoneType' object has no attribute 'full_name'
Comment 1 Simon Fraser (smfr) 2016-10-14 20:26:30 PDT
Created attachment 291701 [details]
Patch
Comment 2 WebKit Commit Bot 2016-10-14 21:20:58 PDT
Comment on attachment 291701 [details]
Patch

Clearing flags on attachment: 291701

Committed r207370: <http://trac.webkit.org/changeset/207370>
Comment 3 WebKit Commit Bot 2016-10-14 21:21:02 PDT
All reviewed patches have been landed.  Closing bug.