Skip to content

Commit

Permalink
feat: removed web_api as an option, it is now a default part of the b…
Browse files Browse the repository at this point in the history
…uild
  • Loading branch information
Danny Piper committed Nov 29, 2024
1 parent cb1c943 commit 29b79be
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 15 deletions.
21 changes: 9 additions & 12 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -272,18 +272,15 @@ target_include_directories(mtg-search-engine PUBLIC ${CMAKE_CURRENT_BINARY_DIR}
add_executable(mtg-search-engine-cli mse/main.c)
target_link_libraries(mtg-search-engine-cli ${LIBS} mtg-search-engine)

option(MSE_WEB_API OFF)
if(MSE_WEB_API)
add_executable(mtg-search-engine-web-api ${WEB_API})
target_link_libraries(
mtg-search-engine-web-api
${LIBS}
crypto
mbedtls
mbedcrypto
mbedx509
mtg-search-engine)
endif()
add_executable(mtg-search-engine-web-api ${WEB_API})
target_link_libraries(
mtg-search-engine-web-api
${LIBS}
crypto
mbedtls
mbedcrypto
mbedx509
mtg-search-engine)

# Setup unit tests
if(UNIX)
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ USER app
RUN mkdir build
WORKDIR /app/build

RUN cmake -DMSE_WEB_API=ON -DUSE_JEMALLOC=ON ..
RUN cmake -DUSE_JEMALLOC=ON ..
RUN cmake --build . -j

FROM base AS app
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,8 @@ cd build

# ./build/
# You can use -DMSE_DOXYGEN=ON to generate documentation
cmake -DMSE_WEB_API=ON -DUSE_JEMALLOC=ON ..
# cmake -DMSE_WEB_API=ON -DUSE_JEMALLOC=ON ..
cmake -DUSE_JEMALLOC=ON ..
# cmake -DUSE_JEMALLOC=ON ..
cmake --build . -j

ctest -V -j # Runs all the tests
Expand Down

0 comments on commit 29b79be

Please sign in to comment.