-
Notifications
You must be signed in to change notification settings - Fork 24
Remove StreamObj and use Pin<Box<dyn Stream>> instead #20
Comments
Github won't let me access my fork at the moment ("The backend storage is temporarily offline."), but I prepared a patch to check that the |
The I thought the semver rule was to keep the exact specified version of a dependency when talking about pre-release/alpha/beta (i.e. |
It looks like Github has my fork working now, so I can push the patch I prepared and open a PR shortly. |
My only dependency is tide In my dependency graph I only have the |
@Kerollmops as a workaround for now you can add an explicit I'm not directly associated with either project so I can't say whether Tide and http-service want to switch to exact versions to avoid this breakage. I'll leave that for the main devs to answer. |
Wow, did not know that. Thank you! TIL something useful and it work! |
According to rust-lang/futures-rs#1352,
StreamObj
is going to be removed in favour forPin<Box<dyn Stream>>
.type BoxStream<'a, T> = Pin<Box<dyn Stream<Item = T> + Send + 'a>>;
The
http-service
is using theStreamObj
for itsBody
:https://github.com/rustasync/http-service/blob/master/src/lib.rs#L29
The text was updated successfully, but these errors were encountered: