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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Install the library files and headers with CMake #77

Closed
matheusgomes28 opened this issue Apr 11, 2023 · 5 comments
Closed

Install the library files and headers with CMake #77

matheusgomes28 opened this issue Apr 11, 2023 · 5 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@matheusgomes28
Copy link
Owner

Need to install the library with CMake, so users can call cmake install

Suggested structure:

  • ${INSTALL_DIR}/lib holds the library files (.a, .lib). 馃摉
  • ${INSTALL_DIR}/include holds the public header files. 馃帶

Relevant video: Deep CMake for library writers.
@matheusgomes28 matheusgomes28 added help wanted Extra attention is needed good first issue Good for newcomers labels Apr 11, 2023
@ayushmaanshrotriya
Copy link

like to work on it

@matheusgomes28
Copy link
Owner Author

like to work on it

@ayushmaanshrotriya sounds good! Let me know if you need help but it should just be about using the install(...) cmake command on the base64pp target defined in base64pp/CMakeLists.txt

@ayushmaanshrotriya
Copy link

some help is needed though

@matheusgomes28
Copy link
Owner Author

some help is needed though

@ayushmaanshrotriya you're basically going to use the Cmake "install" function on the target base64pp . Take a look at the link but you will need to install the lib with the LIBRARY option, and the header file with the FILE option.

@matheusgomes28
Copy link
Owner Author

@ayushmaanshrotriya I'm gonna pick this up as I need it for a different task. Let me know if you've done anything on this and I could wait

@matheusgomes28 matheusgomes28 mentioned this issue Apr 21, 2023
Merged
@matheusgomes28 matheusgomes28 mentioned this issue Apr 22, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@matheusgomes28 @ayushmaanshrotriya