Date: Fri, 20 Apr 2007 10:25:03 -0700 From: Nate Lawson <nate@root.org> To: Yar Tikhiy <yar@comp.chem.msu.su> Cc: Dag-Erling Sm?rgrav <des@des.no>, freebsd-current@freebsd.org Subject: Re: libfetch ftp patch for less latency Message-ID: <4628F76F.80608@root.org> In-Reply-To: <20070420074423.GA22594@comp.chem.msu.su> References: <460AE39B.4070706@root.org> <86ps6g5759.fsf@dwp.des.no> <4617F563.40502@root.org> <200704181648.46348.jhb@freebsd.org> <20070420074423.GA22594@comp.chem.msu.su>
next in thread | previous in thread | raw e-mail | index | archive | help
Yar Tikhiy wrote: > 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@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. > I'll do the work if this is acceptable. -- Nate
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4628F76F.80608>