Remi Rampin

Results 490 comments of Remi Rampin

scp is supposed to show those, see `_recv_confirm()`. Do you get a different error?

You mean that the transfer succeeds either way (scp returns 0) but it prints this message? You can try `scp.channel.recv_stderr()`

Ok I don't know how this message is sent then, sorry.

In practice there is no cost to creating `SCPClient`s over and over, so you can put that in your loop. But I agree.

> FYI: If you wanted to support 2, 3, and 4 argument callbacks with a single argument, it should be possible to use inspect.signature to determine the number of arguments...

This looks good but I had assumed from #94 that we wanted to write into existing file objects, that the caller provides, rather than in-memory ones. I am not sure...

Please stop sending PRs over and over. You can update an existing one.

I'm worried that this will ignore too many errors, not just the "non-regular file" errors. At the minimum this should be configurable, and it would be great to have tests.

The frontend was completely rewritten since, so the pull request is no longer relevant. But the issue might be :wink: