Merge remote-tracking branch 'origin/vrtsim-compilation-fix' into integration_2026_w04 (!3872)
Fix compilation issues with vrtsim taps client Fix two issues regarding vrtsims taps client implementation: - Incorrect #define used to enable taps socket handling - flatbuffers package name changing between ubuntu versions
This commit is contained in:
@@ -51,7 +51,10 @@ RUN apt-get update && \
|
|||||||
m4 \
|
m4 \
|
||||||
# python3-pip for conf template generation
|
# python3-pip for conf template generation
|
||||||
python3-pip \
|
python3-pip \
|
||||||
wget && \
|
wget \
|
||||||
|
flatbuffers-compiler \
|
||||||
|
libflatbuffers-dev \
|
||||||
|
libnanomsg-dev && \
|
||||||
# --break-system-packages allows externally managed environment in pip
|
# --break-system-packages allows externally managed environment in pip
|
||||||
pip3 install --ignore-installed pyyaml --break-system-packages
|
pip3 install --ignore-installed pyyaml --break-system-packages
|
||||||
|
|
||||||
|
|||||||
@@ -49,6 +49,7 @@ RUN \
|
|||||||
--build-lib "telnetsrv enbscope uescope nrscope" \
|
--build-lib "telnetsrv enbscope uescope nrscope" \
|
||||||
-w USRP -t Ethernet \
|
-w USRP -t Ethernet \
|
||||||
--build-e2 --cmake-opt -DKPM_VERSION=$KPM_VERSION --cmake-opt -DE2AP_VERSION=$E2AP_VERSION \
|
--build-e2 --cmake-opt -DKPM_VERSION=$KPM_VERSION --cmake-opt -DE2AP_VERSION=$E2AP_VERSION \
|
||||||
|
--cmake-opt -DOAI_VRTSIM_TAPS_CLIENT=ON \
|
||||||
--noavx512 \
|
--noavx512 \
|
||||||
--build-tool-opt -k10 \
|
--build-tool-opt -k10 \
|
||||||
--cmake-opt -DCMAKE_C_FLAGS="-Werror" --cmake-opt -DCMAKE_CXX_FLAGS="-Werror" $BUILD_OPTION && \
|
--cmake-opt -DCMAKE_C_FLAGS="-Werror" --cmake-opt -DCMAKE_CXX_FLAGS="-Werror" $BUILD_OPTION && \
|
||||||
|
|||||||
@@ -64,7 +64,8 @@ RUN apt-get update && \
|
|||||||
python3-six \
|
python3-six \
|
||||||
python3-requests \
|
python3-requests \
|
||||||
libusb-1.0-0 \
|
libusb-1.0-0 \
|
||||||
libyaml-cpp-dev && \
|
libyaml-cpp-dev \
|
||||||
|
libnanomsg5 && \
|
||||||
# if the --sanitize option was used to build, additional packages are required
|
# if the --sanitize option was used to build, additional packages are required
|
||||||
/bin/bash -c 'if [[ "$BUILD_OPTION" = "--sanitize" ]]; then DEBIAN_FRONTEND=noninteractive apt-get install --yes \
|
/bin/bash -c 'if [[ "$BUILD_OPTION" = "--sanitize" ]]; then DEBIAN_FRONTEND=noninteractive apt-get install --yes \
|
||||||
libasan8 \
|
libasan8 \
|
||||||
|
|||||||
@@ -65,7 +65,8 @@ RUN apt-get update && \
|
|||||||
iproute2 \
|
iproute2 \
|
||||||
iperf3 \
|
iperf3 \
|
||||||
iperf \
|
iperf \
|
||||||
libyaml-cpp-dev && \
|
libyaml-cpp-dev \
|
||||||
|
libnanomsg5 && \
|
||||||
# if the --sanitize option was used to build, additional packages are required
|
# if the --sanitize option was used to build, additional packages are required
|
||||||
/bin/bash -c 'if [[ "$BUILD_OPTION" = "--sanitize" ]]; then DEBIAN_FRONTEND=noninteractive apt-get install --yes \
|
/bin/bash -c 'if [[ "$BUILD_OPTION" = "--sanitize" ]]; then DEBIAN_FRONTEND=noninteractive apt-get install --yes \
|
||||||
libasan8 \
|
libasan8 \
|
||||||
|
|||||||
@@ -26,6 +26,14 @@ if (OAI_VRTSIM_TAPS_CLIENT)
|
|||||||
|
|
||||||
add_library(taps_client taps_client.cpp ${TAPS_API_HEADER})
|
add_library(taps_client taps_client.cpp ${TAPS_API_HEADER})
|
||||||
target_include_directories(taps_client PUBLIC .)
|
target_include_directories(taps_client PUBLIC .)
|
||||||
find_package(Flatbuffers REQUIRED)
|
find_package(FlatBuffers QUIET)
|
||||||
|
if(NOT FlatBuffers_FOUND)
|
||||||
|
# Fall back to lowercase (Ubuntu 22.04)
|
||||||
|
find_package(Flatbuffers REQUIRED)
|
||||||
|
set(FLATBUFFERS_TARGET flatbuffers)
|
||||||
|
else()
|
||||||
|
set(FLATBUFFERS_TARGET flatbuffers::flatbuffers)
|
||||||
|
endif()
|
||||||
|
target_link_libraries(taps_client PUBLIC ${FLATBUFFERS_TARGET})
|
||||||
target_link_libraries(taps_client PUBLIC flatbuffers taps_api SIMU nanomsg)
|
target_link_libraries(taps_client PUBLIC flatbuffers taps_api SIMU nanomsg)
|
||||||
endif()
|
endif()
|
||||||
|
|||||||
@@ -189,13 +189,13 @@ static void vrtsim_readconfig(vrtsim_state_t *vrtsim_state)
|
|||||||
} else {
|
} else {
|
||||||
AssertFatal(false, "Invalid role configuration\n");
|
AssertFatal(false, "Invalid role configuration\n");
|
||||||
}
|
}
|
||||||
#ifdef ENABLE_TAPS_CLIENT
|
#ifdef OAI_VRTSIM_TAPS_CLIENT
|
||||||
if (vrtsim_state->taps_socket) {
|
if (vrtsim_state->taps_socket) {
|
||||||
LOG_A(HW, "VRTSIM: will use taps socket %s\n", vrtsim_state->taps_socket);
|
LOG_A(HW, "VRTSIM: will use taps socket %s\n", vrtsim_state->taps_socket);
|
||||||
}
|
}
|
||||||
#else
|
#else
|
||||||
if (vrtsim_state->taps_socket) {
|
if (vrtsim_state->taps_socket) {
|
||||||
AssertFatal(false, "Invalid configuration: Build with ENABLE_TAPS_CLIENT to use taps socket\n");
|
AssertFatal(false, "Invalid configuration: Build with OAI_VRTSIM_TAPS_CLIENT to use taps socket\n");
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user