Skip to content

Commit ae8c32e

Browse files
cynthiajoanCynthia Jiang
andauthored
Make pod header public so unity can build (#701)
* Make pod header public so unity can build * add comments Co-authored-by: Cynthia Jiang <[email protected]>
1 parent e961980 commit ae8c32e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

ios_pod/CMakeLists.txt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,9 @@ function(setup_pod_headers target_name)
8181
# Add pod name twice as some frameworks moved includes inside a folder with
8282
# the pod name
8383
target_include_directories(${target_name}
84-
PRIVATE
84+
# Link the Pods publicly, so that dependent libraries can access the necessary header files as well.
85+
# Same for the below workaround.
86+
PUBLIC
8587
${FIREBASE_POD_HEADER_DIR}/${pod}
8688
${FIREBASE_POD_HEADER_DIR}/${pod}/${pod}
8789
)
@@ -90,7 +92,7 @@ function(setup_pod_headers target_name)
9092
# we need to include the top-level Private directory as a header path. This only affects Firestore at the moment, as
9193
# no other C++ library includes a private Pod header.
9294
target_include_directories(${target_name}
93-
PRIVATE
95+
PUBLIC
9496
${FIREBASE_POD_PRIVATE_HEADER_DIR}/
9597
${FIREBASE_POD_PRIVATE_HEADER_DIR}/${pod}
9698
${FIREBASE_POD_PRIVATE_HEADER_DIR}/${pod}/${pod}

0 commit comments

Comments
 (0)