Bug 68822 - Add a way to allow adding cc's or messages to bugs based on patch contents.
Summary: Add a way to allow adding cc's or messages to bugs based on patch contents.
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 Levin
URL:
Keywords:
Depends on: 68823 68850 68869 68871 68904 68917 68950 68972 68973 68974 68975 68976 68991 69031 69288 69303 69484 69486 69487
Blocks:
  Show dependency tree / graph
 
Reported: 2011-09-26 11:58 PDT by David Levin
Modified: 2011-10-19 15:54 PDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Levin 2011-09-26 11:58:43 PDT 
The general idea is to have a file which file and/or code regexs and based on what matches add a cc or message to a bug.
Comment 1 David Levin 2011-09-26 12:08:03 PDT 
Here's an example file to give an idea of the syntax:

{
    "DEFINITIONS": {
        "ThreadingFiles": {
            "filename": (r"Source/JavaScriptCore/wtf/RefCountedThreadSafe\.h"
                         r"|Source/WebCore/platform/CrossThreadCopier\.h"),
        },
        "ThreadingStructures": {
            "added": r"RefCountedThreadSafe|deprecatedTurnOffVerifier|AtomicallyInitializedStatic",
        },
        "deprecatedTurnOffVerifier": {
            "added": r"deprecatedTurnOffVerifier",
        },
    },
    "CC_RULES": {
        "ThreadingFiles|ThreadingStructures": [ "levin@chromium.org", ],
    },
    "CC_MESSAGES": {
        "deprecatedTurnOffVerifier": [ "Whoa there grasshopper, don't use deprecatedTurnOffVerifier. It has deprecated in the method name for a reason!", ]
    }

}

The file will be at Tools/Scripts/webkitpy/common/config/watch_list
Comment 2 David Levin 2011-09-30 18:10:03 PDT 
Address https://bugs.webkit.org/show_bug.cgi?id=68973#c16 before resolving.
Comment 3 David Levin 2011-10-19 15:54:57 PDT 
This is now launched -- https://wiki.webkit.org/wiki/WatchList