mirror of
https://github.com/google/nomulus.git
synced 2025-04-29 19:47:51 +02:00
60540fd4bf
* Allow java-format to use java from the PATH When invoking java from the google-java-format-git-diff.sh script, if there is no JAVA_HOME environment variable, attempt to instead run the java binary that is on the PATH. This also adds a few checks to verify that a java binary is available in one of those locations and that the version discovered is Java 11 (which we know to be compatible with the google-java-format jar). Tested: - unset JAVA_HOME, verified that we get the version on the PATH - Set JAVA_HOME to an invalid directory, verified that we get an error. - Changed the "which" command to lookup an nonexistent binary, unset JAVA_HOME and verified that we get a "java not found" error. - Changed the path to point to an old version of java, verified that we get a "bad java version" error. - Verified that the script still runs normally.
134 lines
4.1 KiB
Bash
Executable file
134 lines
4.1 KiB
Bash
Executable file
#!/bin/bash
|
|
# Copyright 2019 The Nomulus Authors. All Rights Reserved.
|
|
#
|
|
# 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.
|
|
#
|
|
# This script applies Google Java format to modified regions in Java source
|
|
# files in a Git repository. It assumes that the repository has a 'master'
|
|
# branch that is only used for merging and is never directly worked on.
|
|
#
|
|
# If invoked on the master branch, this script will format the modified lines
|
|
# relative to HEAD. Otherwise, it uses the
|
|
# 'git merge-base --fork-point origin/master' command to find the latest
|
|
# fork point from origin/master, and formats the modified lines between
|
|
# the fork point and the HEAD of the current branch.
|
|
#
|
|
# Background: existing code base does not conform to Google Java format. Since
|
|
# the team wants to keep the 'blame' feature (find last modifier of a line)
|
|
# usable, we do not want to reformat existing code.
|
|
|
|
set -e
|
|
|
|
USAGE="
|
|
$(basename "$0") [--help] check|format|show
|
|
Incrementally format modified java lines in Git.
|
|
|
|
where:
|
|
--help show this help text
|
|
check check if formatting is necessary
|
|
format format files in place
|
|
show show the effect of the formatting as unified diff"
|
|
|
|
SCRIPT_DIR="$(realpath $(dirname $0))"
|
|
JAR_NAME="google-java-format-1.8-all-deps.jar"
|
|
|
|
# Locate the java binary.
|
|
if [ -n "$JAVA_HOME" ]; then
|
|
JAVA_BIN="$JAVA_HOME/bin/java"
|
|
if [ ! -x "$JAVA_BIN" ]; then
|
|
echo "No java binary found in JAVA_HOME (JAVA_HOME is $JAVA_HOME)"
|
|
exit 1
|
|
fi
|
|
else
|
|
# Use java from the path.
|
|
JAVA_BIN="$(which java)" || JAVA_BIN=""
|
|
if [ -z "$JAVA_BIN" ]; then
|
|
echo "JAVA_HOME is not defined and java was not found on the path"
|
|
exit 1
|
|
fi
|
|
fi
|
|
|
|
if ! "$JAVA_BIN" -version 2>&1 | grep 'version "11\.' >/dev/null; then
|
|
echo "Bad java version. Requires java 11, got:"
|
|
"$JAVA_BIN" -version
|
|
exit 1
|
|
fi
|
|
|
|
function showNoncompliantFiles() {
|
|
local forkPoint="$1"
|
|
local message="$2"
|
|
|
|
git diff -U0 ${forkPoint} | \
|
|
${SCRIPT_DIR}/google-java-format-diff.py \
|
|
--google-java-format-jar "${SCRIPT_DIR}/${JAR_NAME}" \
|
|
-p1 | awk -v "message=$message" \
|
|
'/\+\+\+ ([^ ]*)/ { print message $2 }' 1>&2
|
|
}
|
|
|
|
function callGoogleJavaFormatDiff() {
|
|
local forkPoint
|
|
forkPoint=$(git merge-base origin/master HEAD)
|
|
|
|
local callResult
|
|
case "$1" in
|
|
"check")
|
|
showNoncompliantFiles "$forkPoint" "\033[1mNeeds formatting: "
|
|
callResult=$(git diff -U0 ${forkPoint} | \
|
|
${SCRIPT_DIR}/google-java-format-diff.py \
|
|
--java-binary "$JAVA_BIN" \
|
|
--google-java-format-jar "${SCRIPT_DIR}/${JAR_NAME}" \
|
|
-p1 | wc -l)
|
|
;;
|
|
"format")
|
|
showNoncompliantFiles "$forkPoint" "\033[1mReformatting: "
|
|
callResult=$(git diff -U0 ${forkPoint} | \
|
|
${SCRIPT_DIR}/google-java-format-diff.py \
|
|
--java-binary "$JAVA_BIN" \
|
|
--google-java-format-jar "${SCRIPT_DIR}/${JAR_NAME}" \
|
|
-p1 -i)
|
|
;;
|
|
"show")
|
|
callResult=$(git diff -U0 ${forkPoint} | \
|
|
${SCRIPT_DIR}/google-java-format-diff.py \
|
|
--java-binary "$JAVA_BIN" \
|
|
--google-java-format-jar "${SCRIPT_DIR}/${JAR_NAME}" \
|
|
-p1)
|
|
;;
|
|
esac
|
|
echo -e "\033[0m" 1>&2
|
|
echo "${callResult}"
|
|
exit 0
|
|
}
|
|
|
|
function isJavaFormatNeededOnDiffs() {
|
|
local modifiedLineCount
|
|
modifiedLineCount=$(callGoogleJavaFormatDiff "check")
|
|
|
|
if [[ ${modifiedLineCount} -ne 0 ]]; then
|
|
echo "true"
|
|
else
|
|
echo "false"
|
|
fi
|
|
exit 0
|
|
}
|
|
|
|
# The main function of this script:
|
|
if [[ $# -eq 1 && $1 == "check" ]]; then
|
|
isJavaFormatNeededOnDiffs
|
|
elif [[ $# -eq 1 && $1 == "format" ]]; then
|
|
callGoogleJavaFormatDiff "format"
|
|
elif [[ $# -eq 1 && $1 == "show" ]]; then
|
|
callGoogleJavaFormatDiff "show"
|
|
else
|
|
echo "${USAGE}"
|
|
fi
|