> I meant that the one-time check in AbstractSSHHost._initialize could issue a
> more visible INFO message (as opposed to DEBUG) if rsync is not executable
> on the client and, therefore, disabled for future transfers.

I don't really want this on the tty's stdout, it's a fairly trivial message,
so DEBUG seems more appropriate, or INFO if that's below the cutline.
It's not something we've had frequent (if ever) problems with.

>> Moreover, what you're fixing is to log that fact once, not every time we
>> try to do a transfer ;-)
>
> You're also fixing the performance impact with trying, unsuccessfully, rsync
> for every file transfer.

Aah, excellent point.
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to