Summary: | [git-webkit] Forward unknown args to log and blame | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||||
Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aakash_jain, andresg_22, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Jonathan Bedard
2021-08-20 11:34:50 PDT
Created attachment 436011 [details]
Patch
Comment on attachment 436011 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=436011&action=review > Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/__init__.py:87 > + parsed = parser.parse_args(args=args) Open to other thoughts here, but in general, if the command in question doesn't have args, we want to parse it normally. If it does have args, we want to treat all unknown values as args. The trouble is, we can't figure out which case we're in until we parse the command in the first place. Created attachment 436014 [details]
Patch
Comment on attachment 436014 [details]
Patch
rs=me
Created attachment 436029 [details]
Patch for landing
Committed r281339 (240756@main): <https://commits.webkit.org/240756@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 436029 [details]. |