From 55b5e425818847b410cb29b1694166ce292ec6eb Mon Sep 17 00:00:00 2001 From: Patrice Arruda Date: Fri, 7 Aug 2020 17:55:23 +0000 Subject: [PATCH] Set the OS name in the metrics protobuf file There are cases where the developer is not building a product so the metadata information is not being set. For now, set the OS name so we can track which OS the user is using it. Fixes: b/163118212 Test: m nothing and checked the blueprint Change-Id: I9b521eee2d266b50625aa8e916ddaacb4e2feaea Merged-In: I9b521eee2d266b50625aa8e916ddaacb4e2feaea --- ui/metrics/metrics.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ui/metrics/metrics.go b/ui/metrics/metrics.go index 39b55cb91..8ae085cdf 100644 --- a/ui/metrics/metrics.go +++ b/ui/metrics/metrics.go @@ -17,6 +17,7 @@ package metrics import ( "io/ioutil" "os" + "runtime" "time" "android/soong/ui/metrics/metrics_proto" @@ -94,8 +95,6 @@ func (m *Metrics) SetMetadataMetrics(metadata map[string]string) { m.metrics.HostArch = m.getArch(v) case "HOST_2ND_ARCH": m.metrics.Host_2NdArch = m.getArch(v) - case "HOST_OS": - m.metrics.HostOs = proto.String(v) case "HOST_OS_EXTRA": m.metrics.HostOsExtra = proto.String(v) case "HOST_CROSS_OS": @@ -137,6 +136,7 @@ func (m *Metrics) Serialize() (data []byte, err error) { // exports the output to the file at outputPath func (m *Metrics) Dump(outputPath string) (err error) { + m.metrics.HostOs = proto.String(runtime.GOOS) data, err := m.Serialize() if err != nil { return err