Skip to content

Commit

Permalink
Final fixes for Q1 tag (#90)
Browse files Browse the repository at this point in the history
* Fix typo in incubator demo help text

* Find python3 if only cmake 3.11 is installed

* Fix cameraDemo packaging pybind11 dependency

Install from pip instead of apt to get the required pybind11 >= 2.2.

Co-authored-by: Cameron Evans <[email protected]>
  • Loading branch information
daxhaw and Cameron Evans authored May 23, 2020
1 parent acdee42 commit b88c306
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 2 deletions.
2 changes: 1 addition & 1 deletion demos/incubator/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,7 @@ int main(int argc, const char **argv) {
simulation_thread[0].join();
printf("Simulation stopped...\n");
} else {
printf("Simulatin is not started.\n");
printf("Simulation is not started.\n");
}
break;
case 'l':
Expand Down
11 changes: 11 additions & 0 deletions production/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,17 @@ set(GAIA_PROD_BUILD "${CMAKE_BINARY_DIR}")
message(STATUS "GAIA_INC=${GAIA_INC}")

find_package(Python3 COMPONENTS Interpreter Development)
if (NOT Python3_FOUND)
# Work with CMAKE 3.11 python find package. Looks like FindPython3
# FindPython2 or FindPython work with CMAKE 3.12 and above.
find_package(PythonInterp 3)
find_package(PythonLibs 3)
if (PYTHONINTERP_FOUND AND PYTHONLIBS_FOUND)
set(Python3_FOUND TRUE)
set(Python3_EXECUTABLE "${PYTHON_EXECUTABLE}")
endif()
endif()

find_package(Java)
find_package(JNI)

Expand Down
4 changes: 3 additions & 1 deletion scratch/gregory/cameraDemo/packaging/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,9 @@ RUN wget -O ${APP_RUN} https://github.com/AppImage/AppImageKit/releases/download
# Install Gaia Platform production build dependencies.
RUN apt-get install -y \
clang-8 \
pybind11-dev
python3-pip

RUN pip3 install pybind11

# Build Gaia Platform production.
COPY /production /GaiaPlatform/production
Expand Down

0 comments on commit b88c306

Please sign in to comment.