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

Ability to customize the library's C++ namespace (default is lbstd) #15

Closed
2 tasks done
leha-bot opened this issue Sep 12, 2022 · 0 comments · Fixed by #16
Closed
2 tasks done

Ability to customize the library's C++ namespace (default is lbstd) #15

leha-bot opened this issue Sep 12, 2022 · 0 comments · Fixed by #16
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@leha-bot
Copy link
Member

leha-bot commented Sep 12, 2022

via CMake's configure argument -DLBSTD_VERSION_NS=<custom_namespace>
The macro for name of namespace will be LB_STD_VERSION_NS (with underscore, inconsistent, yes).
Another tasks:

  • Passthrough the CMake arg to C++ #define;
  • Test in tests/CMakeLists.txt for checking the baked C++ namespace's name with baked into CMake;
@leha-bot leha-bot added the enhancement New feature or request label Sep 12, 2022
@leha-bot leha-bot added this to the 0.0.x milestone Sep 12, 2022
@leha-bot leha-bot self-assigned this Sep 12, 2022
leha-bot added a commit that referenced this issue Sep 23, 2023
Via defining the LB_STD_VERSION_NS macro using the
target_compile_definitions().
Its name a bit inconsistent though, but looks better (in my opinion) in
C++, but doesn't look OK in a bunch of CMake configure parameters as it
conflicts with another flags (`LBSTD_VERSION_xxx` vs `LB_STD_VERSION_xxx`).
leha-bot added a commit that referenced this issue Sep 23, 2023
Via defining the LB_STD_VERSION_NS macro using the
target_compile_definitions().
Its name a bit inconsistent though, but looks better (in my opinion) in
C++, but doesn't look OK in a bunch of CMake configure parameters as it
conflicts with another flags (`LBSTD_VERSION_xxx` vs `LB_STD_VERSION_xxx`).
leha-bot added a commit that referenced this issue Sep 23, 2023
Via defining the LB_STD_VERSION_NS macro using the
target_compile_definitions().
Its name a bit inconsistent though, but looks better (in my opinion) in
C++, but doesn't look OK in a bunch of CMake configure parameters as it
conflicts with another flags (`LBSTD_VERSION_xxx` vs `LB_STD_VERSION_xxx`).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant