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

Disable streaming by default #770

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open
9 changes: 8 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,14 @@ option(NATS_BUILD_EXAMPLES "Build examples" ON)
option(NATS_BUILD_LIBUV_EXAMPLE "Build libuv examples" OFF)
option(NATS_BUILD_LIBEVENT_EXAMPLE "Build libevent examples" OFF)
option(NATS_BUILD_STATIC_EXAMPLES "Statically link examples" OFF)
option(NATS_BUILD_STREAMING "Build NATS Streaming" ON)

# Streaming is deprecated, but we still support it for now.
# We will remove it in the future. If you are starting a new project,
# please use JetStream. Another reason for disabling this is that
# building with streaming requires protobuf-c and building on Windows
# is not as straightforward as building without it.
option(NATS_BUILD_STREAMING "Build NATS Streaming" OFF)

option(NATS_BUILD_NO_PREFIX_CONNSTS "No prefix for connection status enum" OFF)
option(NATS_BUILD_LIB_STATIC "Build static library" ON)
option(NATS_BUILD_LIB_SHARED "Build shared library" ON)
Expand Down
2 changes: 1 addition & 1 deletion buildOnTravis.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ if [ "$1" != "gcc" ]; then
fi
fi

cmake .. $3
cmake .. $3 -DNATS_BUILD_STREAMING=ON
res=$?
if [ $res -ne 0 ]; then
exit $res
Expand Down
Loading