Skip site navigation (1)Skip section navigation (2)
Date:      20 Jun 2000 20:02:36 +0200
From:      Dag-Erling Smorgrav <des@flood.ping.uio.no>
To:        "Jordan K. Hubbard" <jkh@zippy.osd.bsdi.com>
Cc:        Wes Peters <wes@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/lib/libfetch ftp.c
Message-ID:  <xzpn1kgs01v.fsf@flood.ping.uio.no>
In-Reply-To: "Jordan K. Hubbard"'s message of "Tue, 20 Jun 2000 07:04:31 -0700"
References:  <6113.961509871@localhost>

next in thread | previous in thread | raw e-mail | index | archive | help
"Jordan K. Hubbard" <jkh@zippy.osd.bsdi.com> writes:
> > Wes Peters <wes@FreeBSD.org> writes:
> > >   Log:
> > >   Add implementation of fetchListFTP.
> > Absolutely not acceptable under any pretext.
> You should also say why.  It does not foster good relations between
> developers when you simply back something out without previous comment
> or any explanation as to why it's not acceptable.

1) I am the maintainer of this code, and the README clearly states
   that although contributions are welcome, no commits should be made
   without my prior agreement.

2) The patch is unnecessarily bulky and duplicates code found
   elsewhere in libfetch. I chose to set it aside until I was finished
   rewriting the HTTP code, which already suffers severely from code
   duplication.

DES
-- 
Dag-Erling Smorgrav - des@flood.ping.uio.no


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?xzpn1kgs01v.fsf>