Skip to content

Commit

Permalink
Fix name of sensor-graph module.
Browse files Browse the repository at this point in the history
Apparently include fails silently, so this was just absent from the
build.

The warning flag change is only needed now because we weren't previously
building non-default (debug) configurations. Now we are, and that
problem has always been present.
  • Loading branch information
DanAlbert committed May 9, 2024
1 parent 87b1ece commit 1667d98
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion sensor-graph/accelerometer/src/main/cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
cmake_minimum_required(VERSION 3.4.1)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Werror")
# TODO: Remove -Wno-unused-variable
# That warning is ignore for now because those are release config only issues
# caused by assert(). The fix is to migrate to CHECK/DCHECK, but that requires
# importing absl or libbase first.
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Werror -Wno-unused-variable")

add_library(accelerometergraph SHARED
sensorgraph.cpp)
Expand Down
2 changes: 1 addition & 1 deletion settings.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ include(":prefab:prefab-dependency:app")
include(":prefab:prefab-publishing:mylibrary")
include(":san-angeles:app")
include(":sanitizers:app")
include(":sensor-graph:app")
include(":sensor-graph:accelerometer")
include(":teapots:choreographer-30fps")
include(":teapots:classic-teapot")
include(":teapots:image-decoder")
Expand Down

0 comments on commit 1667d98

Please sign in to comment.