Skip to content

Commit

Permalink
zephyr: Allow direct CMake inclusions
Browse files Browse the repository at this point in the history
When including these libraries without Kconfigs, CMake was throwing an
error. These changes allow adding direct dependencies in CMake.

Change-Id: Ib36b6af404c506c1742924122ad94f09254b42de
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/175477
Commit-Queue: Yuval Peress <peress@google.com>
Reviewed-by: Taylor Cramer <cramertj@google.com>
  • Loading branch information
yperess authored and CQ Bot Account committed Oct 13, 2023
1 parent c83ead5 commit c30e6cf
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 0 additions & 4 deletions pw_interrupt_zephyr/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,6 @@

include($ENV{PW_ROOT}/pw_build/pigweed.cmake)

if(NOT CONFIG_PIGWEED_INTERRUPT_CONTEXT)
return()
endif()

pw_add_library(pw_interrupt_zephyr.context INTERFACE
HEADERS
public/pw_interrupt_zephyr/context_inline.h
Expand Down
4 changes: 2 additions & 2 deletions pw_sync_zephyr/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ pw_add_library(pw_sync_zephyr.thread_notification_backend INTERFACE
public
public_overrides/thread_notification
PUBLIC_DEPS
pw_sync.binary_semaphore.facade
pw_sync.binary_semaphore
pw_sync.thread_notification.facade
)
pw_zephyrize_libraries_ifdef(
Expand All @@ -96,7 +96,7 @@ pw_add_library(pw_sync_zephyr.timed_thread_notification_backend INTERFACE
public
public_overrides/timed_thread_notification
PUBLIC_DEPS
pw_sync.binary_semaphore.facade
pw_sync.binary_semaphore
pw_sync.timed_thread_notification.facade
)

Expand Down

0 comments on commit c30e6cf

Please sign in to comment.