Skip to content

Commit

Permalink
change logic to true=1 in build script (#409)
Browse files Browse the repository at this point in the history
Closes: #395
To be merged after bytedeco/javacpp-presets#1420
  • Loading branch information
jbkyang-nvi committed Oct 17, 2023
1 parent e028bd2 commit 210f5e8
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/java-api-bindings/scripts/install_dependencies_and_build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ JAVACPP_BRANCH=${JAVACPP_BRANCH:="https://github.com/bytedeco/javacpp-presets.gi
JAVACPP_BRANCH_TAG=${JAVACPP_BRANCH_TAG:="master"}
CMAKE_VERSION=${CMAKE_VERSION:="3.21.1"}
export JAR_INSTALL_PATH="/workspace/install/java-api-bindings"
# Note: Following bashscript convention where True/success == 0 and False == 1
export INCLUDE_DEVELOPER_TOOLS_SERVER=1
# Note: True != 0 and False == 0
export INCLUDE_DEVELOPER_TOOLS_SERVER=0
KEEP_BUILD_DEPENDENCIES=1

for OPTS; do
Expand Down Expand Up @@ -100,7 +100,7 @@ for OPTS; do
shift 2
;;
--enable-developer-tools-server)
export INCLUDE_DEVELOPER_TOOLS_SERVER=0
export INCLUDE_DEVELOPER_TOOLS_SERVER=1
echo "Including developer tools server C++ bindings"
;;
--keep-build-dependencies)
Expand All @@ -111,7 +111,7 @@ for OPTS; do
done
set -x

if [ ${INCLUDE_DEVELOPER_TOOLS_SERVER} -eq 0 ]; then
if [ ${INCLUDE_DEVELOPER_TOOLS_SERVER} -ne 0]; then
# install cmake and rapidjson
apt-get update && apt-get install -y gpg wget && \
wget -O - https://apt.kitware.com/keys/kitware-archive-latest.asc 2>/dev/null | \
Expand All @@ -138,7 +138,7 @@ git clone --single-branch --depth=1 -b ${JAVACPP_BRANCH_TAG} ${JAVACPP_BRANCH}
cd javacpp-presets

# Remove developer_tools/server related build
if [ ${INCLUDE_DEVELOPER_TOOLS_SERVER} -eq 1 ]; then
if [ ${INCLUDE_DEVELOPER_TOOLS_SERVER} -eq 0 ]; then
rm -r tritonserver/src/gen
rm tritonserver/src/main/java/org/bytedeco/tritonserver/presets/tritondevelopertoolsserver.java
fi
Expand Down

0 comments on commit 210f5e8

Please sign in to comment.