-
Notifications
You must be signed in to change notification settings - Fork 19
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
Split up into multiple subcrates #106
Comments
Good question. Here is an initial assorted list of thoughts:
|
Oh, we do have quite different opinions indeed 🙈 I'm careful with dependencies, and I'd trade a dependency the size of nix for half a dozen trivial unsafe calls any time 🙂 I'd not try to get rid of serde or nom for sure, but if I can get rid of nix with some trivial libc wrappers I think it's a positive change 🙂 I believe that reducing dependencies helps; if I use but a very small subset of a dependency (as we do with nix here) I tend to prefer copying selective parts over bringing in a huge amount of unused code, which nonetheless adds to build times. I hate to pay for transitive dependencies of parts I don't even use (e.g. nom and serde if all I use is I also tend to believe that splitting into a workspace of loosly coupled (I mean the I think we'll probably need to agree to disagree here 🙈 If you like we can close this issue; I'll probably just copy the relevant parts of this crate to my projects then (which is totally fine with me, after all the journal interface is pretty much set in stone and I don't expect much changes there 🙂 ) |
@lucab I'm closing this issue, because I don't think we'll be doing it 😄 and I'm just clearing out my issues list 😇 |
Hey @lucab,
this crate became quite big; it has accumulated a few different features already, and got quite a few dependencies.
How about splitting it up into multiple subcrates which allows downstream users to pick only parts of this crate to reduce the amount of dependencies?
Specifically I'd very much like to move the contents of
logging
into a separate crate to avoid pulling in e.g. nom just for writing to the systemd journal, and we could perhaps also join forces with the author of sd_notify or just reexport sd_notify.What do you think?
The text was updated successfully, but these errors were encountered: