async-email / async-smtp

Apache License 2.0
60 stars 13 forks source link

Bump nom dependency to ^7.0 #51

Closed JuxhinDB closed 2 years ago

JuxhinDB commented 2 years ago

Updating nom to latest version as it's causing build errors in twistrs. Tests are passing bar one, however it seems to be related to some missing file/config.

test result: ok. 25 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/transport_file.rs (target/debug/deps/transport_file-4c3d8b1858f4d79e)

running 1 test
test test::file_transport ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/transport_sendmail.rs (target/debug/deps/transport_sendmail-35c9ebf37f972ebe)

running 1 test
test test::sendmail_transport_simple ... FAILED

failures:

---- test::sendmail_transport_simple stdout ----
Err(Io(Os { code: 2, kind: NotFound, message: "No such file or directory" }))
thread 'test::sendmail_transport_simple' panicked at 'assertion failed: result.is_ok()', tests/transport_sendmail.rs:22:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
link2xt commented 2 years ago

Tests are passing bar one, however it seems to be related to some missing file/config.

Probably you don't have sendmail installed?

JuxhinDB commented 2 years ago

Tests are passing bar one, however it seems to be related to some missing file/config.

Probably you don't have sendmail installed?

That was it. :-)

JuxhinDB commented 2 years ago

@link2xt Any update on merging this perhaps?

link2xt commented 2 years ago

cc @dignifiedquire Seems it is a subset of #52 now

link2xt commented 2 years ago

52 is merged