Skip to content

Commit 7bfff6e

Browse files
greenc-FNALCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot <[email protected]>
1 parent 5566be8 commit 7bfff6e

File tree

3 files changed

+4
-2
lines changed

3 files changed

+4
-2
lines changed

.github/workflows/cmake-build.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ on:
88
push:
99
branches: [ main, develop ]
1010
schedule:
11-
- cron: '07 18 * * 6'
11+
- cron: '7 18 * * 6'
1212
workflow_dispatch:
1313
inputs:
1414
ref:

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ add_subdirectory(plugins)
197197
# Link Perfetto to phlex executable if enabled
198198
if(ENABLE_PERFETTO AND TARGET phlex)
199199
target_link_libraries(phlex PRIVATE Perfetto::Perfetto)
200-
target_compile_definitions(phlex PRIVATE PERFETTO_ENABLE_TRACING=1)
200+
201201
endif()
202202

203203
if(PHLEX_USE_FORM)

Modules/FindPerfetto.cmake

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,11 @@ find_file(
2020
find_package_handle_standard_args(Perfetto REQUIRED_VARS Perfetto_INCLUDE_DIR Perfetto_SOURCE)
2121

2222
if(Perfetto_FOUND AND NOT TARGET Perfetto::Perfetto)
23+
find_package(Threads REQUIRED)
2324
add_library(perfetto_impl STATIC "${Perfetto_SOURCE}")
2425
target_include_directories(perfetto_impl PUBLIC "${Perfetto_INCLUDE_DIR}")
2526
target_compile_definitions(perfetto_impl PUBLIC PERFETTO_ENABLE_TRACING=1)
27+
target_link_libraries(perfetto_impl PUBLIC Threads::Threads)
2628
add_library(Perfetto::Perfetto ALIAS perfetto_impl)
2729
endif()
2830

0 commit comments

Comments
 (0)