Re: libfetch ftp patch for less latency

On Wed, Apr 18, 2007 at 04:48:45PM -0400, John Baldwin wrote:
> On Saturday 07 April 2007 15:47, Nate Lawson wrote:
> > Dag-Erling Sm?rgrav wrote:
> > > Nate Lawson <nate_at_root.org> writes:
> > >> Obviously, it's easier to do nothing than something.  So here are some
> > >> options:
> > >>
> > >> 1. Add my patch -- if a server returns an error, I see no way it would
> > >> have changed the PWD.  If you say "CD GARBAGE", what reasonable system
> > >> would return an error and change to some random dir?
> > >>
> > >> 2. Add an env variable (similar to FTP_PASSIVE_MODE, say
> > >> "FTP_SINGLE_CWD") which forces the current behavior.  If not set, fetch
> > >> tries the multi-method first, falls back to the single-method on error.
> > > 
> > > No.
> > > 
> > > Thanks,
> > > 
> > > DES
> > 
> > I forgot:
> > 
> > 3. #ifdef (on or off by default)
> > 
> > Also, can I hear from anyone else besides Mr. No?
> 
> I'm hestitant to make fetch explicitly not follow the RFC.  At the least it
> should follow the RFC by default.  Having it not follow the RFC actually
> broke stuff at work until I fixed it.

I believe that the proposed feature should be conditional on the
TVFS extension in the server (RFC 3659) as it indeed violates the
basic FTP protocol.  OTOH, TVFS seems to provide guarantees that
a single CWD will work as expected.

-- 
Yar