From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Kost Subject: Re: [PATCH] New package: Seren Date: Fri, 06 May 2016 11:15:56 +0300 Message-ID: <87mvo3eh4z.fsf@gmail.com> References: <1462458843.23626.5.camel@gmail.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:45010) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ayaw0-0003EL-Od for guix-devel@gnu.org; Fri, 06 May 2016 04:16:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ayavo-00057X-TH for guix-devel@gnu.org; Fri, 06 May 2016 04:16:23 -0400 Received: from mail-lf0-x241.google.com ([2a00:1450:4010:c07::241]:35342) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ayavn-000531-Dw for guix-devel@gnu.org; Fri, 06 May 2016 04:16:16 -0400 Received: by mail-lf0-x241.google.com with SMTP id u64so12567072lff.2 for ; Fri, 06 May 2016 01:16:01 -0700 (PDT) In-Reply-To: <1462458843.23626.5.camel@gmail.com> (Francesco Frassinelli's message of "Thu, 05 May 2016 16:34:03 +0200") 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: Francesco Frassinelli Cc: guix-devel@gnu.org Francesco Frassinelli (2016-05-05 17:34 +0300) wrote: > Hello, Hello and welcome! > I'm a new Guix user and I created a package for Seren, a P2P VoIP > conference program which uses Opus as codec. You made a new "seren.scm" file, but didn't add a standard header (with copyright, license, etc.) to it. Also new files should be registered in "/gnu/local.mk". However, there is no need to put this pakcage into a separate module. I think (gnu packages telephony) is a suitable module for it. The commit message should look like this. --8<---------------cut here---------------start------------->8--- gnu: Add seren. * gnu/packages/telephony.scm (seren): New variable. --8<---------------cut here---------------end--------------->8--- > From ecffe90a83f8ae0dddef49cfb115e28b854b5d62 Mon Sep 17 00:00:00 2001 > From: Francesco Frassinelli > Date: Thu, 5 May 2016 16:23:15 +0200 > Subject: [PATCH] Add package Seren > > --- > gnu/packages/seren.scm | 46 ++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 46 insertions(+) > create mode 100644 gnu/packages/seren.scm > > diff --git a/gnu/packages/seren.scm b/gnu/packages/seren.scm > new file mode 100644 > index 0000000..8ac95ef > --- /dev/null > +++ b/gnu/packages/seren.scm > @@ -0,0 +1,46 @@ > +(define-module (gnu packages seren) > + #:use-module (guix packages) > + #:use-module (guix download) > + #:use-module (guix build-system gnu) > + #:use-module (guix licenses) > + #:use-module (gnu packages linux) > + #:use-module (gnu packages xiph) > + #:use-module (gnu packages ncurses) > + #:use-module (gnu packages multiprecision)) > + > +(define-public seren > + (package > + (name "seren") > + (version "0.0.21") > + (source (origin > + (method url-fetch) > + (uri (string-append "http://holdenc.altervista.org/" > + "seren/downloads/seren-" version > + ".tar.gz")) > + (sha256 > + (base32 > + "06mams6bng7ib7p2zpfq88kdr4ffril9svzc9lprkb0wjgmkglk9")))) > + (build-system gnu-build-system) > + (arguments > + `(#:phases > + (modify-phases %standard-phases > + (delete 'check)))) Usually instead of deleting 'check' phase, we disable tests, so it could look like this: (arguments '(#:tests? #f)) ; no "check" target > + (inputs `(("alsa-lib" ,alsa-lib) > + ("opus" ,opus) > + ("libogg" ,libogg) > + ("ncurses" ,ncurses) > + ("gmp" ,gmp))) > + (synopsis "Simple VoIP program to create conferences from the terminal") > + (description > + "Seren is a simple VoIP program based on the Opus codec that allows you > +to create a voice conference from the terminal, with up to 10 > +partecipants, without having to register accounts, exchange emails, or > +add people to contact lists. > +All you need to join an existing conference is the host name or IP > +address of one of the partecipants. > +Seren creates a dynamic peer-to-peer network of equivalent nodes which > +exchange text and audio data using a UDP connection, and offers the > +user the ability to change the quality/bitrate on the fly, encrypt the > +traffic and record the calls.") If you want to make separate paragraphs here, use additional newlines: (description "Seren is a simple VoIP program based on the Opus codec that allows you to create a voice conference from the terminal, with up to 10 partecipants, without having to register accounts, exchange emails, or add people to contact lists. All you need to join an existing conference is the host name or IP address of one of the partecipants. Seren creates a dynamic peer-to-peer network of equivalent nodes which exchange text and audio data using a UDP connection, and offers the user the ability to change the quality/bitrate on the fly, encrypt the traffic and record the calls.") (Though I think this description is a bit long, but it's probably OK) > + (home-page "http://holdenc.altervista.org/seren/") > + (license gpl3+))) The rest looks good to me, could you send an updated patch? And thanks for contributing! -- Alex