Skip to content

Conversation

@Bill-hbrhbr
Copy link
Collaborator

@Bill-hbrhbr Bill-hbrhbr commented Jan 27, 2025

Description

  • Add a complete boilerplate CMake project with a dummy header-only interface library and a dummy executable that calls the library.
  • Implement custom CMake command cpp_library to simplify the CMakeLists.txt of each individual ystdlib-cpp lib
  • Added more exclusion paths to CMake gersmi linters

Does not support installing ystdlib into system paths yet.

Validation performed

  • The project build succeeds and the executable can run successfully.

@Bill-hbrhbr Bill-hbrhbr marked this pull request as ready for review January 27, 2025 18:49
@Bill-hbrhbr Bill-hbrhbr requested a review from davidlion January 27, 2025 18:51
@davidlion davidlion marked this pull request as draft January 27, 2025 18:52
@Bill-hbrhbr Bill-hbrhbr marked this pull request as ready for review January 27, 2025 23:06
@Bill-hbrhbr Bill-hbrhbr changed the title feat: Add CMake features for managing interface libraries of ystdlib-cpp. feat: Add CMake features for building interface libraries of ystdlib-cpp. Jan 27, 2025
@Bill-hbrhbr Bill-hbrhbr requested a review from davidlion February 1, 2025 04:48
@Bill-hbrhbr Bill-hbrhbr requested a review from davidlion February 3, 2025 08:45
@davidlion davidlion changed the title feat: Add CMake features for building interface libraries of ystdlib-cpp. feat: Add CMakeLists.txt files and helper function for building interface libraries. Feb 3, 2025
Bill-hbrhbr and others added 3 commits February 3, 2025 04:23
@Bill-hbrhbr Bill-hbrhbr requested a review from davidlion February 3, 2025 09:31
davidlion
davidlion previously approved these changes Feb 3, 2025
@Bill-hbrhbr Bill-hbrhbr requested a review from davidlion February 3, 2025 09:38
@Bill-hbrhbr Bill-hbrhbr merged commit b92875f into y-scope:main Feb 3, 2025
6 checks passed
@Bill-hbrhbr Bill-hbrhbr deleted the header-only branch February 3, 2025 18:47
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