Skip to content

Install static library#37

Open
srcejon wants to merge 1 commit into
dnwrnr:masterfrom
srcejon:android
Open

Install static library#37
srcejon wants to merge 1 commit into
dnwrnr:masterfrom
srcejon:android

Conversation

@srcejon

@srcejon srcejon commented Nov 10, 2023

Copy link
Copy Markdown
Contributor

Currently the static library that is built doesn't get installed. This PR installs it.

Comment thread libsgp4/CMakeLists.txt

add_library(sgp4 STATIC ${SRCS} ${INCS})
add_library(sgp4s SHARED ${SRCS} ${INCS})
install( TARGETS sgp4 LIBRARY DESTINATION lib )

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ARCHIVE type is for static libraries:

Suggested change
install( TARGETS sgp4 LIBRARY DESTINATION lib )
install( TARGETS sgp4 ARCHIVE DESTINATION lib )

https://cmake.org/cmake/help/v3.13/command/install.html#installing-targets

@jeaogom jeaogom mentioned this pull request Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants