Skip to content

Expose the include directory of the bundled sqlite version#1441

Merged
gwenn merged 1 commit intorusqlite:masterfrom
GiGainfosystems:feature/expose_include_dir
Jan 25, 2024
Merged

Expose the include directory of the bundled sqlite version#1441
gwenn merged 1 commit intorusqlite:masterfrom
GiGainfosystems:feature/expose_include_dir

Conversation

@weiznich
Copy link
Contributor

This commit exposes the include directory of the bundled sqlite version via by instructing cargo to set the DEP_SQLITE3_INCLUDE enviroment variable as documented in the cargo book:
https://doc.rust-lang.org/cargo/reference/build-script-examples.html#using-another-sys-crate

This enables other sys crates to rely on the bundled sqlite version and use it as base for their compilation if they have a (sub-)dependency on libsqlite3.

This commit exposes the include directory of the bundled sqlite version
via by instructing cargo to set the `DEP_SQLITE3_INCLUDE` enviroment
variable as documented in the cargo book:
https://doc.rust-lang.org/cargo/reference/build-script-examples.html#using-another-sys-crate

This enables other sys crates to rely on the bundled sqlite version and
use it as base for their compilation if they have a (sub-)dependency on
libsqlite3.
@gwenn
Copy link
Collaborator

gwenn commented Jan 25, 2024

@weiznich
Copy link
Contributor Author

As that's a feature of build.rs files you would need to search for lang:rust and not for lang:toml, right?

https://github.com/search?q=lang%3ARust++%22cargo%3Ainclude%3D%22&type=code

Or is that trying to say: Noone is using libsqlite3-sys in that way yet?

@gwenn
Copy link
Collaborator

gwenn commented Jan 25, 2024

As that's a feature of build.rs files you would need to search for lang:rust and not for lang:toml, right?

Sorry, you are right.
And I guess we can ignore the code coverage error.

@gwenn gwenn merged commit 42a82fb into rusqlite:master Jan 25, 2024
@weiznich
Copy link
Contributor Author

Thanks for the fast response ❤️

@gwenn
Copy link
Collaborator

gwenn commented Jan 27, 2024

libsqlite3-sys v0.28.0 released

@weiznich
Copy link
Contributor Author

Thanks ❤️

@gwenn gwenn mentioned this pull request Feb 16, 2024
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