Gordon Ross wrote:

> Now the more interesting stuff:
> 
> Some time ago, I observed that the Connectathon NFS tests
> would often terminate early after any test error.

Hi Gordon,

I am mostly happy with your changes; I think they're a
useful addition to a test suite that gets intermittent
love :-)

One thing - the special tests used to copy themselves to
the target and run from there, and you have changed it to
at least not always do that.  I understand why we need a
way to avoid executing from the target remote filesystem,
but executing that way has at least a time or two shown
off bugs, mostly in the client, because of the way exec()
manages mappings of the binary.  I would prefer that this
continue to be the default mode of operation to maintain
this coverage, and to have an option to have the special
tests do a chdir() instead.  This is not huge, but I'd
rather not lose the regression coverage.

Rob T

Reply via email to