From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark H Weaver Subject: Re: [PATCH] Add yaggo. Date: Sun, 05 Jul 2015 03:33:59 -0400 Message-ID: <87vbdz3vo8.fsf@netris.org> References: <558A3377.6060301@uq.edu.au> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:57280) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZBeRU-000051-Dv for guix-devel@gnu.org; Sun, 05 Jul 2015 03:34:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZBeRP-00056V-Gb for guix-devel@gnu.org; Sun, 05 Jul 2015 03:34:24 -0400 Received: from world.peace.net ([50.252.239.5]:60632) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZBeRP-00056P-DN for guix-devel@gnu.org; Sun, 05 Jul 2015 03:34:19 -0400 In-Reply-To: <558A3377.6060301@uq.edu.au> (Ben Woodcroft's message of "Wed, 24 Jun 2015 14:35:03 +1000") 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: Ben Woodcroft Cc: guix-devel@gnu.org Ben Woodcroft writes: > From d220bdeac89660108de96a96107daf89182310e0 Mon Sep 17 00:00:00 2001 > From: Ben Woodcroft > Date: Wed, 24 Jun 2015 14:32:26 +1000 > Subject: [PATCH] gnu: Add yaggo. > > * gnu/packages/ruby.scm (yaggo): New variable. > --- > gnu/packages/ruby.scm | 32 ++++++++++++++++++++++++++++++++ > 1 file changed, 32 insertions(+) > > diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm > index af87cf5..19006d3 100644 > --- a/gnu/packages/ruby.scm > +++ b/gnu/packages/ruby.scm > @@ -3,6 +3,7 @@ > ;;; Copyright =C2=A9 2014 Ludovic Court=C3=A8s > ;;; Copyright =C2=A9 2014, 2015 Mark H Weaver > ;;; Copyright =C2=A9 2014 David Thompson > +;;; Copyright =C2=A9 2015 Ben Woodcroft > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -697,3 +698,34 @@ options and parsing command line flags.") > net/http library.") > (home-page "https://github.com/nicksieger/multipart-post") > (license license:expat))) > + > +(define-public yaggo > + (package > + (name "yaggo") > + (version "1.5.4") > + (source (origin > + (method url-fetch) > + (uri (string-append > + "https://github.com/gmarcais/yaggo/archive/v" > + version ".tar.gz")) > + (file-name (string-append name "-" version ".tar.gz")) > + (sha256 > + (base32 > + "1mxfvrim03xg80agws9zdpk00r0kjpqhw3xbli0w8wvsnsa274y3")))) Please change the tabs to spaces. > + (build-system ruby-build-system) > + (arguments > + `(#:tests? #f ;; no rake test, and Makefile in test/ appears malform= ed Just one semicolon there, please. Our convention, borrowed from , is to use one semicolon for "margin comments", i.e. comments to the right of the code, and two semicolons for comments on their own line preceding code. It's made more important by the fact that formatting commands in Emacs Lisp editing modes will act differently depending on how many semicolon are present. > + #:phases > + (modify-phases %standard-phases > + (replace 'build Please align the "(" under the "o" in "modify-phases" > + (lambda* _ You can use plain 'lambda' here. > + (zero? (system* "rake" > + "gem"))))))) It would look better to move "gem" to the preceeding line. Better yet, how about putting the entire 'replace' on one line, like this: (modify-phases %standard-phases (replace 'build (lambda _ (zero? (system* "rake" "gem"))))))) Normally I use more whitespace, but in this case I think it's perfectly readable. What do you think? > + (synopsis "Generate C++ command line parsers using getopt_long") > + (description "Yaggo is a tool to generate command line parsers for > +C++. Yaggo stands for 'Yet Another GenGetOpt' and is inspired by GNU > +Gengetopt. It reads a configuration file describing the switches and ar= gument > +for a C++ program and it generates one header file that parses the comma= nd > +line using getopt_long(3).") > + (home-page "https://github.com/gmarcais/yaggo") > + (license license:gpl3+))) Can you send an updated patch? Thanks, Mark