aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Collingbourne <pcc@google.com>2024-03-26 10:56:07 -0700
committerPeter Collingbourne <pcc@google.com>2024-03-26 10:56:39 -0700
commit46b68970517885557cf25f21dc5b9b4c802155ee (patch)
tree18d8606266c6e0034966ecc50154fc03ca212712
parent3c14f6566f7605da2d79591e9d450f50bddc5756 (diff)
downloadguice-46b68970517885557cf25f21dc5b9b4c802155ee.tar.gz
Pass output file argument to guice_munge instead of redirecting.HEADmastermain
It looks like a recent change to the JDK has caused logs to be written to stdout intermittently, causing errors such as the following when compiling a source file processed by guice_munge: out.cfa64l5/soong/.intermediates/external/guice/guice/linux_glibc_common/javac/srcjars/external/guice/core/src/com/google/inject/internal/InternalProviderInstanceBindingImpl.java:1: error: class, interface, enum, or record expected [0.003s][warning][perf,memops] Cannot use file /tmp/hsperfdata_nobody/75537 because it is locked by another process (errno = 11) ^ Avoid the problem by passing an output file argument to guice_munge instead of redirecting stdout to the output file. Bug: 330410989 Change-Id: I218f9b8597b77350e810d1bfcfa60fe94312c876
-rw-r--r--Android.bp2
1 files changed, 1 insertions, 1 deletions
diff --git a/Android.bp b/Android.bp
index fe9cbf7a..cfc7813c 100644
--- a/Android.bp
+++ b/Android.bp
@@ -108,7 +108,7 @@ genrule {
],
cmd: "for src in $(in); do " +
" mkdir -p $$(dirname $(genDir)/$${src}) && " +
- " $(location guice_munge) -DNO_AOP $${src} > $(genDir)/$${src} || exit 1; " +
+ " $(location guice_munge) -DNO_AOP $${src} $(genDir)/$${src} || exit 1; " +
" done && " +
" $(location soong_zip) -o $(out) -C $(genDir) -D $(genDir)",
}