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

Removal of option to load SQLite library dynamically #61

Closed
utelle opened this issue Jul 7, 2019 · 2 comments
Closed

Removal of option to load SQLite library dynamically #61

utelle opened this issue Jul 7, 2019 · 2 comments

Comments

@utelle
Copy link
Owner

utelle commented Jul 7, 2019

Since wxSQLite3 version 3.5.0 (released in January 2017) the source code of the SQLite library is an integrated part of wxSQLite3 to simplify building the component and to make it easier to keep the features supported by SQLite and the wxSQLite3 wrapper in sync.

Using the symbol wxUSE_DYNAMIC_SQLITE3_LOAD it is currently still possible to select the option to load the SQLite library at runtime (DLL resp shared object). However, this opens the risk of loading a SQLite version that was not compiled with options matching those used for building the wxSQLite3 wrapper.

Therefore it is intended to remove the option to load SQLite at runtime from the next major release of wxSQLite3, unless a significant number of wxSQLite3 users object to this change.

Please share your opinions. Thanks.

@utelle utelle added the question label Jul 7, 2019
@Deamhan
Copy link
Contributor

Deamhan commented Jul 8, 2019

Good idea. I didn't add this option to CMakeLists.txt in view of it.

@utelle
Copy link
Owner Author

utelle commented Aug 4, 2019

The option to dynamically load the SQLite library will be removed in version 5.x of wxSQLite3 (see also issue #72).

@utelle utelle closed this as completed Aug 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants