Add no_std support to diesel #4906
Merged
+1,051
−692
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
weiznich
force-pushed
the
diesel/no_std
branch
4 times, most recently
from
05905e3 to
cf53096
Compare
weiznich
force-pushed
the
diesel/no_std
branch
from
cf53096 to
e02c478
Compare
weiznich
force-pushed
the
diesel/no_std
branch
7 times, most recently
from
f8a6772 to
0bdf1c0
Compare
weiznich
force-pushed
the
diesel/no_std
branch
from
0bdf1c0 to
6ac1983
Compare
Member
Author
|
@LucaCappelletti94 You ask where you could help. This PR could use a review from someone that's not me 😉 |
LucaCappelletti94
suggested changes
Jan 30, 2026
Contributor
LucaCappelletti94
left a comment
LucaCappelletti94
left a comment
There was a problem hiding this comment.
Just a few minor core -> ::core changes needed for consistency
diesel_derives/src/sql_function.rs
Outdated
| ) -> QueryResult<()> | ||
| where | ||
| F: Fn(#(#arg_name,)*) -> Ret + std::panic::UnwindSafe + Send + 'static, | ||
| F: Fn(#(#arg_name,)*) -> Ret + core::panic::UnwindSafe + Send + 'static, |
Contributor
There was a problem hiding this comment.
For consistency this should be ::core I believe
diesel_derives/src/sql_function.rs
Outdated
| ) -> QueryResult<()> | ||
| where | ||
| F: FnMut(#(#arg_name,)*) -> Ret + std::panic::UnwindSafe + Send + 'static, | ||
| F: FnMut(#(#arg_name,)*) -> Ret + core::panic::UnwindSafe + Send + 'static, |
Contributor
There was a problem hiding this comment.
For consistency this should be ::core I believe
diesel_derives/src/sql_function.rs
Outdated
| ) -> QueryResult<()> | ||
| where | ||
| F: Fn() -> Ret + std::panic::UnwindSafe + Send + 'static, | ||
| F: Fn() -> Ret + core::panic::UnwindSafe + Send + 'static, |
Contributor
There was a problem hiding this comment.
For consistency this should be ::core I believe
diesel_derives/src/sql_function.rs
Outdated
| ) -> QueryResult<()> | ||
| where | ||
| F: FnMut() -> Ret + std::panic::UnwindSafe + Send + 'static, | ||
| F: FnMut() -> Ret + core::panic::UnwindSafe + Send + 'static, |
Contributor
There was a problem hiding this comment.
For consistency this should be ::core I believe
diesel_derives/src/sql_function.rs
Outdated
| + std::panic::UnwindSafe | ||
| + std::panic::RefUnwindSafe, | ||
| + core::panic::UnwindSafe | ||
| + core::panic::RefUnwindSafe, |
Contributor
There was a problem hiding this comment.
For consistency these should be ::core I believe
weiznich
force-pushed
the
diesel/no_std
branch
from
6ac1983 to
c1867fb
Compare
weiznich
force-pushed
the
diesel/no_std
branch
from
c1867fb to
a8554fd
Compare
This commit adds support for using diesel with the SQLite backend in a no-std (core + alloc) environment. This mainly means fixing a lot of imports to use the relevant items from core or alloc instead. Notable exceptions are: * There is no hashmap in alloc, so we need to pull in hashbrown for this * There is no panicing/abort support in core so we need our custom shims for these. * Global instrumentation setting is not supported yet (cfged away) as it requires lock which are not supported on embedded targets. This commit also introduces a new `std` default feature for enabling `std` lib support. If disabled the users need to manually enable the `hashbrown` feature to pull in hashbrown for hashmap support. Cargo doesn't support pulling in dependencies for disabled features. Finally this commit also adds a `example/sqlite/embedded` example project that demonstrates how to use this on a esp32-c6.
weiznich
force-pushed
the
diesel/no_std
branch
from
a8554fd to
6ca3911
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit adds support for using diesel with the SQLite backend
in a no-std (core + alloc) environment.
This mainly means fixing a lot of imports to use the relevant items from core or alloc instead. Notable exceptions are:
This commit also introduces a new
stddefault feature for enablingstdlib support. If disabled the users need to manually enable thehashbrownfeature to pull in hashbrown for hashmap support. Cargo doesn't support pulling in dependencies for disabled features.Finally this commit also adds a
example/sqlite/embeddedexample project that demonstrates how to use this on a esp32-c6.This is marked as a draft for now as there are a few upstream things that I would like to resolve properly before merging this: