Skip to content

Commit

Permalink
Merge "Disable goma for javac actions in Make"
Browse files Browse the repository at this point in the history
  • Loading branch information
colincross authored and Gerrit Code Review committed Mar 5, 2020
2 parents 701cb0c + 28ff3e3 commit 074d8e1
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 4 deletions.
3 changes: 2 additions & 1 deletion core/goma.mk
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ ifneq ($(filter-out false,$(USE_GOMA)),)
# use both ccache and gomacc.
CC_WRAPPER := $(strip $(CC_WRAPPER) $(GOMA_CC))
CXX_WRAPPER := $(strip $(CXX_WRAPPER) $(GOMA_CC))
JAVAC_WRAPPER := $(strip $(JAVAC_WRAPPER) $(GOMA_CC))
# b/143658984: goma can't handle the --system argument to javac
#JAVAC_WRAPPER := $(strip $(JAVAC_WRAPPER) $(GOMA_CC))

goma_dir :=
endif
3 changes: 2 additions & 1 deletion core/host_dalvik_java_library.mk
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,8 @@ java_sources_deps := \
$(java_source_list_file): $(java_sources_deps)
$(write-java-source-list)

$(full_classes_compiled_jar): .KATI_NINJA_POOL := $(GOMA_POOL)
# TODO(b/143658984): goma can't handle the --system argument to javac.
#$(full_classes_compiled_jar): .KATI_NINJA_POOL := $(GOMA_POOL)
$(full_classes_compiled_jar): PRIVATE_JAVA_LAYERS_FILE := $(layers_file)
$(full_classes_compiled_jar): PRIVATE_JAVACFLAGS := $(LOCAL_JAVACFLAGS) $(annotation_processor_flags)
$(full_classes_compiled_jar): PRIVATE_JAR_EXCLUDE_FILES :=
Expand Down
3 changes: 2 additions & 1 deletion core/host_java_library.mk
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,8 @@ java_sources_deps := \
$(java_source_list_file): $(java_sources_deps)
$(write-java-source-list)

$(full_classes_compiled_jar): .KATI_NINJA_POOL := $(GOMA_POOL)
# TODO(b/143658984): goma can't handle the --system argument to javac.
#$(full_classes_compiled_jar): .KATI_NINJA_POOL := $(GOMA_POOL)
$(full_classes_compiled_jar): PRIVATE_JAVA_LAYERS_FILE := $(layers_file)
$(full_classes_compiled_jar): PRIVATE_JAVACFLAGS := $(LOCAL_JAVACFLAGS) $(annotation_processor_flags)
$(full_classes_compiled_jar): PRIVATE_JAR_EXCLUDE_FILES :=
Expand Down
3 changes: 2 additions & 1 deletion core/java.mk
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,8 @@ $(eval $(call copy-one-file,$(full_classes_header_jarjar),$(full_classes_header_

endif # TURBINE_ENABLED != false

$(full_classes_compiled_jar): .KATI_NINJA_POOL := $(GOMA_POOL)
# TODO(b/143658984): goma can't handle the --system argument to javac.
#$(full_classes_compiled_jar): .KATI_NINJA_POOL := $(GOMA_POOL)
$(full_classes_compiled_jar): PRIVATE_JAVACFLAGS := $(LOCAL_JAVACFLAGS) $(annotation_processor_flags)
$(full_classes_compiled_jar): PRIVATE_JAR_EXCLUDE_FILES := $(LOCAL_JAR_EXCLUDE_FILES)
$(full_classes_compiled_jar): PRIVATE_JAR_PACKAGES := $(LOCAL_JAR_PACKAGES)
Expand Down

0 comments on commit 074d8e1

Please sign in to comment.