From 667f688f361ed732d4209cc38e73335425cd2e2c Mon Sep 17 00:00:00 2001 From: Steven Moreland Date: Thu, 26 Jul 2018 12:55:08 -0700 Subject: Build Java aidl w/ '-b'. As it turns out, AIDL originally passed and returned success when it was asked to compile an unstructured parcelable even though no output files were created. The build system currently can't handle this because it is expecting outputs here, so I am adding this argument here which will cause the compiler to exit and error earlier as expected. Notice also that the make implementation of the AIDL build system also passes in this argument. Test: m framework (which invokes this many times) Bug: N/A Change-Id: Ia9b5280dc3593756784035a523399545c9947d80 --- java/java.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'java/java.go') diff --git a/java/java.go b/java/java.go index a2ba7e6f9..38c89f4c8 100644 --- a/java/java.go +++ b/java/java.go @@ -631,7 +631,8 @@ func (j *Module) aidlFlags(ctx android.ModuleContext, aidlPreprocess android.Opt aidlIncludes = append(aidlIncludes, android.PathsForSource(ctx, j.deviceProperties.Aidl.Include_dirs)...) - var flags []string + flags := []string{"-b"} + if aidlPreprocess.Valid() { flags = append(flags, "-p"+aidlPreprocess.String()) } else { -- cgit v1.2.3-59-g8ed1b