On Thu, Jun 01, 2017 at 04:31:53PM -0400, Aaron Conole wrote: > Ben Pfaff <[email protected]> writes: > > > ovs_checkpatch_parse() takes 2 arguments, not sys.exit(). Oops. > > > > Fixes: 95bd35d3db19 ("checkpatch: Implement -f option more usefully.") > > Signed-off-by: Ben Pfaff <[email protected]> > > --- > > Weird. > > 04:26:00 aconole {master} ~/git/ovs$ git log --grep="Implement -f" | grep > Date: > Date: Fri May 26 11:22:36 2017 -0700 > 04:26:01 aconole {master} ~/git/ovs$ rm flake8-check > 04:26:07 aconole {master} ~/git/ovs$ make flake8-check >/dev/null && echo > "WORKING" > WORKING > > Just tried doing an actual commit, instead of an am: > > Traceback (most recent call last): > File "utilities/checkpatch.py", line 415, in <module> > sys.exit(ovs_checkpatch_parse(sys.stdin.read()), '-') > TypeError: ovs_checkpatch_parse() takes exactly 2 arguments (1 given) > > :/ > > Sorry I missed it during my review.
No problem, I should have found it myself. > Acked-by: Aaron Conole <[email protected]> I applied this to master. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
