Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change logic to true=1 #409

Merged
merged 2 commits into from
Oct 13, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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/success == 1 and False != 0
jbkyang-nvi marked this conversation as resolved.
Show resolved Hide resolved
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 Down Expand Up @@ -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
Loading