fix oss build failures
Summary: fix for the unbound variable and include fizz header
MODIFY
build_helper.sh
build_helper.sh
@@ -50,7 +50,7 @@ if [ -z "${BUILD_DIR-}" ] ; then
mkdir -p $BUILD_DIR
fi
-if [[ ! -z $MVFST_FOLLY_USE_JEMALLOC ]]; then
+if [[ -n "${MVFST_FOLLY_USE_JEMALLOC-}" ]]; then
if [[ "$MVFST_FOLLY_USE_JEMALLOC" != "n" ]]; then
unset $MVFST_FOLLY_USE_JEMALLOC
fi
@@ -169,7 +169,7 @@ function setup_folly() {
cd "$FOLLY_BUILD_DIR" || exit
# check for environment variable. If
- if [[ -z $MVFST_FOLLY_USE_JEMALLOC ]]; then
+ if [[ -z "${MVFST_FOLLY_USE_JEMALLOC-}" ]]; then
cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo \
-DCMAKE_PREFIX_PATH="$FOLLY_INSTALL_DIR" \
-DCMAKE_INSTALL_PREFIX="$FOLLY_INSTALL_DIR" \

MODIFY
quic/CMakeLists.txt
quic/CMakeLists.txt
@@ -43,6 +43,7 @@ add_library(
target_include_directories(
mvfst_exception PUBLIC
+ $<BUILD_INTERFACE:${LIBFIZZ_INCLUDE_DIR}>
$<BUILD_INTERFACE:${QUIC_FBCODE_ROOT}>
$<INSTALL_INTERFACE:include/>
)
@@ -53,6 +54,11 @@ target_compile_options(
${_QUIC_COMMON_COMPILE_OPTIONS}
)
+add_dependencies(
+ mvfst_exception
+ ${FIZZ_TARGET}
+)
+
target_link_libraries(
mvfst_exception PUBLIC
Folly::folly

fix oss build failures

Summary: fix for the unbound variable and include fizz header

Reviewed By: mjoras

Differential Revision: D15565195

fbshipit-source-id: 1b02955a2546d57da71cab40ad0ceccdf4cee577