AutoRolls include Chromium bug links

go/util/util.go

Index: go/util/util.go
diff --git a/go/util/util.go b/go/util/util.go
index be684a1aaf1a780555f5f8374665cbb84ee870e5..4e99e96491e071424d8869cfbbb463e0cf82ce8b 100644
--- a/go/util/util.go
+++ b/go/util/util.go
@@ -28,6 +28,10 @@ const (
 	TB
 	PB
 
+	PROJECT_CHROMIUM    = "chromium"
+	BUG_DEFAULT_PROJECT = PROJECT_CHROMIUM
+	BUGS_PATTERN        = "(?m)^BUG=(.+)$"
+
 	SECONDS_TO_MILLIS = int64(time.Second / time.Millisecond)
 	MILLIS_TO_NANOS   = int64(time.Millisecond / time.Nanosecond)
 
@@ -39,6 +43,8 @@ const (
 )
 
 var (
+	BUGS_REGEX = regexp.MustCompile(BUGS_PATTERN)
+
 	// randomNameAdj is a list of adjectives for building random names.
 	randomNameAdj = []string{
 		"autumn", "hidden", "bitter", "misty", "silent", "empty", "dry", "dark",
@@ -604,3 +610,28 @@ func ChunkIter(s []int, chunkSize int, fn func([]int) error) error {
 	}
 	return nil
 }
+
+// BugsFromCommitMsg parses BUG= tags from a commit message and returns them.
+func BugsFromCommitMsg(msg string) map[string][]string {
+	rv := map[string][]string{}
+	m := BUGS_REGEX.FindStringSubmatch(msg)
+	if len(m) > 1 {
+		bugs := strings.Split(m[1], ",")
+		for _, b := range bugs {
+			b = strings.Trim(b, " ")
+			split := strings.SplitN(strings.Trim(b, " "), ":", 2)
+			project := BUG_DEFAULT_PROJECT
+			bug := split[0]
+			if len(split) > 1 {
+				project = split[0]
+				bug = split[1]
+			}
+			if rv[project] == nil {
+				rv[project] = []string{}
+			}
+			rv[project] = append(rv[project], bug)
+		}
+	}
+	glog.Errorf("%v", rv)
+	return rv

[rmistry, 2016/04/04 12:45:09]

Why return a dictionary of projects to bugs? It would be simpler and still
correct to just parse out all strings delimited by ',' and return an array of
bugs which might contain contents like ["123", "chromium:123", "skia:123"]. You
can then blindly specify them in --bug.
This way you do not care about needing to make assumptions about what the
default project is, all this code will do is send straight through whatever is
specified as BUG= in the individual CLs.

[borenet, 2016/04/04 15:50:07]

The autoroller only wants to specify chromium bugs.  Others are not relevant to
DEPS rolls.  The caller could scan through and only take the "chromium:123" or
"123" bugs, but it makes more sense to me to do it here.  Plus, I'm doubtful
that the default project will ever change.

+}