Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add cmake configuration #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

keiradiance
Copy link

Add CMakeLists.txt with shared library installation target.

@neov5
Copy link

neov5 commented Jul 11, 2024

tried to use this, but it worked when include_directories(src) was changed to target_include_directories(logc_shared PUBLIC src). Could not import the log.h file otherwise.

@tuaris
Copy link

tuaris commented Aug 30, 2024

I don't understand why people insist on making a shared library. The code is small and simple enough to integrate with your app. Not everything needs to be overly complex and be a shared library.

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.

3 participants