commit 7fd905ae26ac2cb9333a81feb928df339c5b4bc4
author Jeff Gaston <jeffrygaston@google.com> Tue May 23 17:24:53 2017 -0700
committer Jeff Gaston <jeffrygaston@google.com> Wed May 24 13:06:35 2017 -0700
tree de02ef8c49daeb401ad2bdb2463ca4df2dea28ea
parent de4b0463f220c069af1aa10d594b39ceafdfa78b

Run bpfix at the end of androidmk

Bug: 38351765
Test: androidmk Android.mk > Android.bp
Change-Id: Iefcae1b01f4e915ec20782b302ba4350a20d0f1a

androidmk/Android.bp

diff --git a/androidmk/Android.bp b/androidmk/Android.bp
index 67baf0a..442452f 100644
--- a/androidmk/Android.bp
+++ b/androidmk/Android.bp
@@ -29,6 +29,7 @@
     deps: [
         "androidmk-parser",
         "blueprint-parser",
+        "bpfix-lib",
     ],
 }
 

androidmk/cmd/androidmk/androidmk.go

diff --git a/androidmk/cmd/androidmk/androidmk.go b/androidmk/cmd/androidmk/androidmk.go
index 1d94b65..d26643a 100644
--- a/androidmk/cmd/androidmk/androidmk.go
+++ b/androidmk/cmd/androidmk/androidmk.go
@@ -8,6 +8,8 @@
 	"strings"
 	"text/scanner"
 
+	"android/soong/bpfix/bpfix"
+
 	mkparser "android/soong/androidmk/parser"
 
 	bpparser "github.com/google/blueprint/parser"
@@ -176,10 +178,18 @@
 		}
 	}
 
-	out, err := bpparser.Print(&bpparser.File{
+	tree := &bpparser.File{
 		Defs:     file.defs,
 		Comments: file.comments,
-	})
+	}
+
+	// check for common supported but undesirable structures and clean them up
+	fixed, err := bpfix.FixTree(tree, bpfix.NewFixRequest().AddAll())
+	if err != nil {
+		return "", []error{err}
+	}
+
+	out, err := bpparser.Print(fixed)
 	if err != nil {
 		return "", []error{err}
 	}