Skip to content

Commit

Permalink
Fix installation of public headers
Browse files Browse the repository at this point in the history
The header lists must be available in the local scope to be set as
PUBLIC_HEADER target properties as well as in the parent scope for
Doxygen to use.

This regression was caused by 35f5802
  • Loading branch information
leonlynch committed Nov 9, 2024
1 parent 46fa563 commit 586c3c2
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -112,8 +112,8 @@ set(iso7816_HEADERS # PUBLIC_HEADER property requires a list instead of individu
iso7816_apdu.h
iso7816_compact_tlv.h
iso7816_strings.h
PARENT_SCOPE # Doxygen generator requires a list of headers
)
set(iso7816_HEADERS ${iso7816_HEADERS} PARENT_SCOPE) # Doxygen generator requires a list of headers
add_library(emv::iso7816 ALIAS iso7816)
set_target_properties(iso7816
PROPERTIES
Expand Down Expand Up @@ -147,8 +147,8 @@ add_library(iso8825
)
set(iso8825_HEADERS # PUBLIC_HEADER property requires a list instead of individual entries
iso8825_ber.h
PARENT_SCOPE # Doxygen generator requires a list of headers
)
set(iso8825_HEADERS ${iso8825_HEADERS} PARENT_SCOPE) # Doxygen generator requires a list of headers
add_library(emv::iso8825 ALIAS iso8825)
set_target_properties(iso8825
PROPERTIES
Expand Down Expand Up @@ -183,8 +183,8 @@ add_library(iso8859)
add_library(emv::iso8859 ALIAS iso8859)
set(iso8859_HEADERS # PUBLIC_HEADER property requires a list instead of individual entries
iso8859.h
PARENT_SCOPE # Doxygen generator requires a list of headers
)
set(iso8859_HEADERS ${iso8859_HEADERS} PARENT_SCOPE) # Doxygen generator requires a list of headers
set_target_properties(iso8859
PROPERTIES
PUBLIC_HEADER "${iso8859_HEADERS}"
Expand Down Expand Up @@ -265,8 +265,8 @@ set(emv_HEADERS # PUBLIC_HEADER property requires a list instead of individual e
emv_ttl.h
emv_app.h
emv_tal.h
PARENT_SCOPE # Doxygen generator requires a list of headers
)
set(emv_HEADERS ${emv_HEADERS} PARENT_SCOPE) # Doxygen generator requires a list of headers
add_library(emv::emv ALIAS emv)
target_compile_definitions(emv PUBLIC EMV_DEBUG_ENABLED) # Enabled all debugging
target_link_libraries(emv
Expand Down Expand Up @@ -320,8 +320,8 @@ set(emv_strings_HEADERS # PUBLIC_HEADER property requires a list instead of indi
emv_strings.h
isocodes_lookup.h
mcc_lookup.h
PARENT_SCOPE # Doxygen generator requires a list of headers
)
set(emv_strings_HEADERS ${emv_strings_HEADERS} PARENT_SCOPE) # Doxygen generator requires a list of headers
add_library(emv::emv_strings ALIAS emv_strings)
# The EMVSTRINGS_PKGCONFIG_REQ_PRIV and EMVSTRINGS_PKGCONFIG_LIBS_PRIV variables are set
# for the parent scope to facilitate the generation of pkgconfig files.
Expand Down

0 comments on commit 586c3c2

Please sign in to comment.