3ca9298d99
This will allow individual modules to enable error prone checks as needed so that the change to treat an error prone check as a compiler error can be done atomically with the changes to fix the issues that check reveals. Tested with the following steps, cleaning up the targets in between to ensure that it was rebuilt everytime. Set LOCAL_ERROR_PRONE_FLAGS := -Xep:ClassCanBeStatic:WARN compiler error in junit-params-test target. Built javac-check-junit-params-test with RUN_ERROR_PRONE=true to make sure that compilation failed. Built the javac-check-junit-params-test without RUN_ERROR_PRONE set to make sure that the flag is not included unless explicitly requested. Built the junit-params-test as normal to make sure it still worked properly. Bug: 31507496 Test: See above Change-Id: I88b5800dacd0e30dd9dcfa5f42e89d1428957365
118 lines
4.4 KiB
Makefile
118 lines
4.4 KiB
Makefile
#
|
|
# Copyright (C) 2008 The Android Open Source Project
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
#
|
|
|
|
$(call record-module-type,HOST_JAVA_LIBRARY)
|
|
|
|
#
|
|
# Standard rules for building a host java library.
|
|
#
|
|
|
|
#######################################
|
|
include $(BUILD_SYSTEM)/host_java_library_common.mk
|
|
#######################################
|
|
|
|
# Enable emma instrumentation only if the module asks so.
|
|
ifeq (true,$(LOCAL_EMMA_INSTRUMENT))
|
|
ifneq (true,$(EMMA_INSTRUMENT))
|
|
LOCAL_EMMA_INSTRUMENT :=
|
|
endif
|
|
endif
|
|
|
|
full_classes_compiled_jar := $(intermediates.COMMON)/classes-full-debug.jar
|
|
full_classes_jarjar_jar := $(intermediates.COMMON)/classes-jarjar.jar
|
|
emma_intermediates_dir := $(intermediates.COMMON)/emma_out
|
|
# emma is hardcoded to use the leaf name of its input for the output file --
|
|
# only the output directory can be changed
|
|
full_classes_emma_jar := $(emma_intermediates_dir)/lib/$(notdir $(full_classes_jarjar_jar))
|
|
|
|
LOCAL_INTERMEDIATE_TARGETS += \
|
|
$(full_classes_compiled_jar) \
|
|
$(full_classes_jarjar_jar) \
|
|
$(full_classes_emma_jar)
|
|
|
|
#######################################
|
|
include $(BUILD_SYSTEM)/base_rules.mk
|
|
#######################################
|
|
|
|
java_sources := $(addprefix $(LOCAL_PATH)/, $(filter %.java,$(LOCAL_SRC_FILES))) \
|
|
$(filter %.java,$(LOCAL_GENERATED_SOURCES))
|
|
all_java_sources := $(java_sources)
|
|
|
|
include $(BUILD_SYSTEM)/java_common.mk
|
|
|
|
# The layers file allows you to enforce a layering between java packages.
|
|
# Run build/tools/java-layers.py for more details.
|
|
layers_file := $(addprefix $(LOCAL_PATH)/, $(LOCAL_JAVA_LAYERS_FILE))
|
|
|
|
# If error prone is enabled then add LOCAL_ERROR_PRONE_FLAGS to LOCAL_JAVACFLAGS
|
|
ifeq ($(RUN_ERROR_PRONE),true)
|
|
LOCAL_JAVACFLAGS += $(LOCAL_ERROR_PRONE_FLAGS)
|
|
endif
|
|
|
|
$(full_classes_compiled_jar): PRIVATE_JAVA_LAYERS_FILE := $(layers_file)
|
|
$(full_classes_compiled_jar): PRIVATE_JAVACFLAGS := $(GLOBAL_JAVAC_DEBUG_FLAGS) $(LOCAL_JAVACFLAGS)
|
|
$(full_classes_compiled_jar): PRIVATE_JAR_EXCLUDE_FILES :=
|
|
$(full_classes_compiled_jar): PRIVATE_JAR_PACKAGES :=
|
|
$(full_classes_compiled_jar): PRIVATE_JAR_EXCLUDE_PACKAGES :=
|
|
$(full_classes_compiled_jar): \
|
|
$(java_sources) \
|
|
$(java_resource_sources) \
|
|
$(full_java_lib_deps) \
|
|
$(jar_manifest_file) \
|
|
$(proto_java_sources_file_stamp) \
|
|
$(NORMALIZE_PATH) \
|
|
$(LOCAL_ADDITIONAL_DEPENDENCIES)
|
|
$(transform-host-java-to-package)
|
|
|
|
javac-check : $(full_classes_compiled_jar)
|
|
javac-check-$(LOCAL_MODULE) : $(full_classes_compiled_jar)
|
|
|
|
# Run jarjar if necessary, otherwise just copy the file.
|
|
ifneq ($(strip $(LOCAL_JARJAR_RULES)),)
|
|
$(full_classes_jarjar_jar): PRIVATE_JARJAR_RULES := $(LOCAL_JARJAR_RULES)
|
|
$(full_classes_jarjar_jar): $(full_classes_compiled_jar) $(LOCAL_JARJAR_RULES) | $(JARJAR)
|
|
@echo JarJar: $@
|
|
$(hide) java -jar $(JARJAR) process $(PRIVATE_JARJAR_RULES) $< $@
|
|
else
|
|
$(full_classes_jarjar_jar): $(full_classes_compiled_jar) | $(ACP)
|
|
@echo Copying: $@
|
|
$(hide) $(ACP) -fp $< $@
|
|
endif
|
|
|
|
ifeq (true,$(LOCAL_EMMA_INSTRUMENT))
|
|
$(full_classes_emma_jar): PRIVATE_EMMA_COVERAGE_FILE := $(intermediates.COMMON)/coverage.em
|
|
$(full_classes_emma_jar): PRIVATE_EMMA_INTERMEDIATES_DIR := $(emma_intermediates_dir)
|
|
ifdef LOCAL_EMMA_COVERAGE_FILTER
|
|
$(full_classes_emma_jar): PRIVATE_EMMA_COVERAGE_FILTER := $(LOCAL_EMMA_COVERAGE_FILTER)
|
|
else
|
|
# by default, avoid applying emma instrumentation onto emma classes itself,
|
|
# otherwise there will be exceptions thrown
|
|
$(full_classes_emma_jar): PRIVATE_EMMA_COVERAGE_FILTER := *,-emma,-emmarun,-com.vladium.*
|
|
endif
|
|
# this rule will generate both $(PRIVATE_EMMA_COVERAGE_FILE) and
|
|
# $(full_classes_emma_jar)
|
|
$(full_classes_emma_jar) : $(full_classes_jarjar_jar) | $(EMMA_JAR)
|
|
$(transform-classes.jar-to-emma)
|
|
|
|
$(LOCAL_BUILT_MODULE) : $(full_classes_emma_jar)
|
|
@echo Copying: $@
|
|
$(hide) $(ACP) -fp $< $@
|
|
|
|
else # LOCAL_EMMA_INSTRUMENT
|
|
$(LOCAL_BUILT_MODULE) : $(full_classes_jarjar_jar) | $(ACP)
|
|
@echo Copying: $@
|
|
$(hide) $(ACP) -fp $< $@
|
|
endif # LOCAL_EMMA_INSTRUMENT
|