-
Notifications
You must be signed in to change notification settings - Fork 14
CMakeLists: Export the module targets. Document the changes #96
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
Conversation
- Cleanup the code that creates the module targets - Rename the module targets to sqlpp23::xxxxx_module - When exporting module targets also export additional information that allows the targets to be imported.
d63ffd6 to
dd9f6b7
Compare
dd9f6b7 to
11b4326
Compare
|
Fixed a typo in the documentation: specifix -> specific |
docs/setup.md
Outdated
| | sqlpp23::sqlite3 | #include <sqlpp23/sqlite3/sqlite3.h> | The SQLite3 connector as headers | | ||
| | sqlpp23::sqlite3_module | import sqlpp23::sqlite3; | The SQLite3 connector as a module | | ||
|
|
||
| [^1]: The SQLCipher connector re-uses the codebase of the SQLite3 connector. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
foot notes 1 and 2 seem to be swapped?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed I somehow missed that! That should be fixed now.
…scriptions of the module targets.
11b4326 to
689a03f
Compare
|
Thanks for the changes, LGTM! |
|
Great! That concludes pretty much all the changes related to modules, that I had in mind. So right now sqlpp23 is one of the few libraries that fully support modules (modulo any bugs that the recent changes may have introduced :-) |
|
I probably don't say this often enough: Thanks for your continued support! I really appreciate you pushing for it. |
|
@rbock |
This PR exports the CMake module targets and documents them. I built and tested it with
(all connectors included, modules disabled)
and
(all connectors included, modules enabled)
So hopefully it doesn't break anything.
The test targets could be renamed or aliased as well to make their names more in-line with the names of the module targets, but that's not urgent and can wait until someone finds the time to do that :-)