diff options
author | Patrick Georgi <pgeorgi@google.com> | 2018-11-12 17:52:24 +0100 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2018-11-13 17:44:04 +0000 |
commit | 89bd4892b3f0dc2039bf1d02eabce5c7f1825585 (patch) | |
tree | 3e99c51d1a761e3cb6ced762ee8b87ef05dc8c14 | |
parent | 0cadafaac964fd602060658fda5724156f6c64f7 (diff) |
util/scripts/maintainers.go: Provide delimiters between maintainers
Help automated tools make sense of the output.
Instead of "[name 1 <email> name 2 <email>]", it now prints
"name 1 <email>, name 2 <email>". As long as there are no commas in the
maintainer names, they can be split easily.
Change-Id: I4a254f566404b081a08923bc7ceb49f02039aa2a
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/29604
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r-- | util/scripts/maintainers.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/util/scripts/maintainers.go b/util/scripts/maintainers.go index 623562bd64..4d6cb7d822 100644 --- a/util/scripts/maintainers.go +++ b/util/scripts/maintainers.go @@ -20,6 +20,7 @@ import ( "os" "os/exec" "regexp" + "strings" ) type subsystem struct { @@ -151,7 +152,7 @@ func find_maintainer(fname string) { success = true fmt.Println(fname, "is in subsystem", subsystem.name) - fmt.Println("Maintainers: ", subsystem.maintainer) + fmt.Println("Maintainers: ", strings.Join(subsystem.maintainer, ", ")) } } if !success { |