From 4c233ed0042d0eda3cea8f915f6093a50d92b556 Mon Sep 17 00:00:00 2001
From: Andrew Wells <130512013+andrewmwells-amazon@users.noreply.github.com>
Date: Wed, 1 Nov 2023 15:48:00 -0700
Subject: [PATCH] make findbugs a compileOnly dependency (#56)

---
 CedarJava/build.gradle | 4 +++-
 CedarJava/config.sh    | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/CedarJava/build.gradle b/CedarJava/build.gradle
index a86d070a..d380c5a5 100644
--- a/CedarJava/build.gradle
+++ b/CedarJava/build.gradle
@@ -67,11 +67,13 @@ dependencies {
     implementation 'com.fasterxml.jackson.datatype:jackson-datatype-jdk8:2.15.0'
     implementation 'org.slf4j:slf4j-api:2.0.7'
     implementation 'org.apache.commons:commons-text:1.10.0'
-    implementation 'com.google.code.findbugs:findbugs:3.0.1'
     implementation 'org.apache.logging.log4j:log4j-core:2.20.0'
     implementation 'org.apache.logging.log4j:log4j-to-slf4j:2.20.0'
     implementation 'com.google.guava:guava:31.0.1-jre'
 
+    compileOnly 'com.google.code.findbugs:findbugs:3.0.1'
+    testCompileOnly 'com.google.code.findbugs:findbugs:3.0.1'
+
     testImplementation 'org.junit.jupiter:junit-jupiter-params:5.9.3'
     testImplementation 'org.junit.jupiter:junit-jupiter-engine:5.9.3'
     testImplementation 'org.mockito:mockito-inline:5.2.0'
diff --git a/CedarJava/config.sh b/CedarJava/config.sh
index 41cde92b..c06a92a4 100644
--- a/CedarJava/config.sh
+++ b/CedarJava/config.sh
@@ -14,7 +14,7 @@ if [ "$(uname)" == "Darwin" ]; then
 else
     ffi_lib_str="    environment 'CEDAR_JAVA_FFI_LIB', '"$parent_dir"/CedarJavaFFI/target/debug/libcedar_java_ffi.so'"
 fi
-sed "85s;.*;$ffi_lib_str;" "build.gradle" > new_build.gradle
+sed "87s;.*;$ffi_lib_str;" "build.gradle" > new_build.gradle
 mv new_build.gradle build.gradle
 
 # In CI, we need to pull the latest cedar-policy to match the latest cedar-integration-tests
@@ -23,7 +23,7 @@ mv new_build.gradle build.gradle
 # If you call this script with `run_int_tests`, we assume you have `cedar` checkout out in the `cedar-java` dir
 if [ "$#" -ne 0 ] && [ "$1" == "run_int_tests" ]; then
     integration_tests_str="    environment 'CEDAR_INTEGRATION_TESTS_ROOT', '"$parent_dir"/cedar/cedar-integration-tests'"
-    sed "84s;.*;$integration_tests_str;" "build.gradle" > new_build.gradle
+    sed "86s;.*;$integration_tests_str;" "build.gradle" > new_build.gradle
     mv new_build.gradle build.gradle
 
     export MUST_RUN_CEDAR_INTEGRATION_TESTS=1