Update examples

[charleschege]

This project looks promising. I like the simplicity. However, can you kindly update your examples with every release to prevent errors. I have tried a few and some of them result in errors, for example, the UDP and TCP echo server examples result in error error[E0599]: no method named nextfound for typeruntime::net::tcp::Incoming<'_>in the current scope --> src/main.rs:18:46

[charleschege]

The echo server in the description page also shows nothing when using nc -u localhost 8080

[yoshuawuyts]

@iamcharleschege for the UDP Echo example, after you start typing things and hit enter it'll respond. I just tried it standalone, and it seems to work!

[yoshuawuyts]

Also confirmed the TCP Echo example works outside of the repo.

---

I'm not sure what's going on. I've uploaded a repro of the TCP Example here: https://github.com/yoshuawuyts/repro-runtime-13/blob/master/src/main.rs. Everything seems to be working as expected.

The error you're seeing would imply that futures::prelude::* is somehow not importing the Stream trait. I'm not sure why that might be occurring, but perhaps if you share more of your platform details I could help figure out what's going on?

[ZhouHansen]

Encounter the same error, and add more details：

error[E0599]: no method named `next` found for type `runtime::net::tcp::Incoming<'_>` in the current scope
  --> src/main.rs:18:46
   |
18 |     while let Some(stream) = await!(incoming.next()) {
   |                                              ^^^^
   |
   = help: items from traits can only be used if the trait is in scope
help: the following trait is implemented but not in scope, perhaps add a `use` for it:
   |
8  | use futures_util::stream::StreamExt;
   |

Platform:
macOS 10.13.6
node v11.0.0
npm 6.8.0

cargo.toml:

[dependencies]
runtime = "0.3.0-alpha.1"
runtime-native = "0.3.0-alpha.1"
runtime-raw = "0.3.0-alpha.1"
futures = "0.1.26"
rand = "0.6.5"
failure = "0.1.5"

[charleschege]

The example https://github.com/yoshuawuyts/repro-runtime-13/blob/master/src/main.rs gives the same error

[charleschege]

Environment is Debian 9,
Rustc version is Rustc 1.35.0-nightly (aa99abeb2 2019-04-14)

Cargo.toml file

edition = "2018"

[dependencies]
runtime = "0.3.0-alpha.2"
futures = "0.1.26"

[yoshuawuyts]

@ZhouHansen @iamcharleschege Ah! -- could you try using the following in your Cargo.toml:

edition = "2018"

[dependencies]
runtime = "0.3.0-alpha.2"
futures-preview = "0.3.0-alpha.14"

Right now the futures crate still builds for futures 0.1, whereas futures-preview uses futures 0.3. Both can be used in code as use futures; without a problem.

This feels like something we should perhaps mention in our documentation too. Sorry about this!

[ZhouHansen]

It works! and maybe we should mention the dependencies for each example.

Hello World example requires

[dependencies]
runtime-native = "0.3.0-alpha.1"
runtime-raw = "0.3.0-alpha.1"

[charleschege]

This works! Please add to documentation

[yoshuawuyts]

@ZhouHansen the dependency problems should be fixed now in runtime@0.3.0-alpha.2; so all you need is runtime.

We had some problems with our macros that we've since fixed (:

[yoshuawuyts]

#16 should fix this going forward; happy it now all works. Thanks for the feedback! going to close this for now ✨