chore: gate embed p2p transport on cargo feature#157
Conversation
|
What is the plan for chat-cli? Does this change aim to remove the the transport choice within chat-cli and commit to using embedded λDelivery? The existing approach, (supporting both cargo feature declaration as wells as envvars) was needed was to support the current setup of chat-cli, which allowed the delivery mechanism to be chosen at compile time via envvars. |
|
Another PR after this change: I don't see any hard blocker to keep the env feature. |
jazzz
left a comment
There was a problem hiding this comment.
I don't see any hard blocker to keep the env feature.
If you mean that you want to remove the "env feature" from the cli app then I don't see any issue with this change.
I'd further suggest these changes for the future PR.
- removing the '--transport' flag all together. Embedded delivery needs to be compiled anyway. Remove the added complexity and simplify usage.
Logos-deliveryshould be reserved forlogos-deliveryvialogos-core. Useembedded*to refer to linking directly to the compiled library.
41f6577 to
a0a2362
Compare
Refactor on feature usage in embed p2p transport, use a consistent cargo feature, not env-dependent cfg