From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marius Bakke Subject: Re: [PATCH] Add iPerf Date: Tue, 18 Oct 2016 22:37:40 +0100 Message-ID: <87funt72zf.fsf@duckhunt.i-did-not-set--mail-host-address--so-tickle-me> References: <20161018180646.48bbb60d@gondolin.arda> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:50343) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bwc4w-0007iK-Gw for guix-devel@gnu.org; Tue, 18 Oct 2016 17:37:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bwc4s-0007Mq-JW for guix-devel@gnu.org; Tue, 18 Oct 2016 17:37:46 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:50046) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1bwc4s-0007M3-F6 for guix-devel@gnu.org; Tue, 18 Oct 2016 17:37:42 -0400 In-Reply-To: <20161018180646.48bbb60d@gondolin.arda> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Benz Schenk , guix-devel@gnu.org Benz Schenk writes: > Hi > > This is my first patch, it's really straight-forward. > But I'm not quite sure about the naming because there's iperf, iperf2 > and iperf3 which are different projects. The original iperf is > abandoned and iperf2 is backwards compatible and still gets bug fixes > and iperf3, which this patch is for, is in active development and adds a > lot of new features but also lacks some of the options of previous > versions. Thus it's often referred to as iperf3, which is also the name > of the binary, so I'm not sure wheter the package should better be > called iperf3 or iperf. Hi Benz, Thanks for this patch! I think naming it "iperf" is fine, even though the binary and manual is called iperf3. If we need iperf2 later it can be packaged as "iperf@2". I'm not sure about the home page: the manual and github page both list http://software.es.net/iperf/ as the home. What is the relation between iperf.fr and this project? Should we not go with the official source? Other than that I'd like to make the following modifications. This can be done before pushing, no need to send an updated patch: * Change synopsis to "TCP, UDP and SCTP bandwidth measurement tool". * Update license to match what's in the LICENSE file. * Fix minor typos in the description (bandwith->bandwidth). * Change commit message to match our current convention. * Download the tarball from http://downloads.es.net/pub/iperf/ Are you okay with these changes (including the home page)? Thanks again, and welcome! :)