From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Wurmus Subject: Re: New package: ifstatus Date: Mon, 14 Sep 2015 21:25:47 +0200 Message-ID: <87613c951g.fsf@elephly.net> References: <87bnd5dkmh.fsf@xsteve.at> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:60434) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZbZO8-0001O6-1S for guix-devel@gnu.org; Mon, 14 Sep 2015 15:26:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZbZO4-0002sW-N2 for guix-devel@gnu.org; Mon, 14 Sep 2015 15:26:04 -0400 Received: from sender163-mail.zoho.com ([74.201.84.163]:25911) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZbZO4-0002ru-Dm for guix-devel@gnu.org; Mon, 14 Sep 2015 15:26:00 -0400 In-reply-to: <87bnd5dkmh.fsf@xsteve.at> 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-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Stefan =?utf-8?Q?Reich=C3=B6r?= Cc: guix-devel@gnu.org Hello Stefan, thank you for the contribution! I think this would best fit to the module in ‘networking.scm’. Usually, we use git to track changes and create patches from committed changes. You could add your package recipe to ‘networking.scm’ and create a commit with the message: gnu: Add ifstatus. * gnu/packages/networking.scm (ifstatus): New variable. Then you can create a patch from your last commit by running git format-patch -1 and just send it to the mailing list. Now some comments about the package. > ;;; Copyright © 2014 Stefan Reichoer It’s 2015 here ;) > (source > (origin > (method url-fetch) > (uri (string-append "http://ifstatus.sourceforge.net/download/ifstatus-v" > version ".tar.gz")) This line is a little too long. Also, you might be able to use “mirror://sourceforge/” instead of this direct download URL. > (sha256 > (base32 > "045cbsq9ps32j24v8y5hpyqxnqn9mpaf3mgvirlhgpqyb9jsia0c")) > (modules '((guix build utils))) > (snippet > '(begin > (substitute* "Main.h" > (("#include ") "#include \n#include ")))) > )) This is not indented as it should be. How about this instead: (snippet '(substitute* "Main.h" (("#include ") "#include \n#include "))) > (arguments > '(#:tests? #f ;no "check" target > #:phases > (modify-phases %standard-phases > (delete 'configure) ;; no configure script > (replace 'install > (lambda* (#:key outputs #:allow-other-keys) > (let* ((out (assoc-ref outputs "out")) > (bin (string-append out "/bin"))) > (mkdir-p bin) > (copy-file "ifstatus" > (string-append bin "/ifstatus")))))))) Also here the indentation is off. Please align the opening parentheses of the ‘(delete ...)’ and ‘(replace ...)’ under the ‘o’ of ‘(modify-phases ...)’. Furthermore, please only use one semicolon for margin comments like “; no configure script”. Two semicolons are used only for full line comments. > (description > "IFStatus was developed for Linux users that are usually in console mode. > It is a simple, easy to use program for displaying commonly needed / wanted statistics > in real time about ingoing and outgoing traffic of multiple network interfaces that is > usually hard to find, with a simple and effecient view. It is the substitute for > PPPStatus and EthStatus projects.") “Linux” users? ;) s/effecient/efficient/ Please use double-spacing between sentences. How about something like this instead: “IFStatus is a simple, easy-to-use program for displaying commonly needed / wanted real-time traffic statistics of multiple network interfaces, with a simple and efficient view on the command line. It is intended as a substitute for the PPPStatus and EthStatus projects.” ~~ Ricardo