From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark H Weaver Subject: Re: [PATCH] gnu: Add dnsmasq. Date: Sun, 01 Feb 2015 01:18:05 -0500 Message-ID: <87egqaf942.fsf@netris.org> References: <87twz6r7q7.fsf@taylan.uni.cx> 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]:59761) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YHnrK-0006vO-2a for guix-devel@gnu.org; Sun, 01 Feb 2015 01:18:15 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YHnrG-0006iG-PD for guix-devel@gnu.org; Sun, 01 Feb 2015 01:18:14 -0500 Received: from world.peace.net ([50.252.239.5]:57690) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YHnrG-0006iC-Jy for guix-devel@gnu.org; Sun, 01 Feb 2015 01:18:10 -0500 In-Reply-To: <87twz6r7q7.fsf@taylan.uni.cx> ("\"Taylan Ulrich \=\?utf-8\?Q\?\=5C\=22Bay\=C4\=B1rl\=C4\=B1\=2FKammer\=5C\=22\=5C\=22\=22's\?\= message of "Sat, 31 Jan 2015 21:54:56 +0100") 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: Taylan Ulrich =?utf-8?Q?=22Bay=C4=B1rl=C4=B1?= =?utf-8?Q?=2FKammer=22?= Cc: guix-devel@gnu.org taylanbayirli@gmail.com ("Taylan Ulrich "Bay=C4=B1rl=C4=B1/Kammer\"") write= s: > From f943afbfcdbe78a91064844609391373c845b498 Mon Sep 17 00:00:00 2001 > From: =3D?UTF-8?q?Taylan=3D20Ulrich=3D20Bay=3DC4=3DB1rl=3DC4=3DB1/Kammer?= =3D > > Date: Sat, 31 Jan 2015 21:51:08 +0100 > Subject: [PATCH 2/2] gnu: Add dnsmasq. > > * gnu-system.am (GNU_SYSTEM_MODULES): Add dnsmasq.scm. > * gnu/packages/dnsmasq.scm: New file. > --- > gnu-system.am | 1 + > gnu/packages/dnsmasq.scm | 58 ++++++++++++++++++++++++++++++++++++++++++= ++++++ > 2 files changed, 59 insertions(+) > create mode 100644 gnu/packages/dnsmasq.scm > > diff --git a/gnu-system.am b/gnu-system.am > index 706ad57..b81a42f 100644 > --- a/gnu-system.am > +++ b/gnu-system.am > @@ -79,6 +79,7 @@ GNU_SYSTEM_MODULES =3D \ > gnu/packages/dejagnu.scm \ > gnu/packages/dictionaries.scm \ > gnu/packages/disk.scm \ > + gnu/packages/dnsmasq.scm \ > gnu/packages/docbook.scm \ > gnu/packages/doxygen.scm \ > gnu/packages/dwm.scm \ > diff --git a/gnu/packages/dnsmasq.scm b/gnu/packages/dnsmasq.scm > new file mode 100644 > index 0000000..7303b9f > --- /dev/null > +++ b/gnu/packages/dnsmasq.scm > @@ -0,0 +1,58 @@ > +;;; GNU Guix --- Functional package management for GNU > +;;; Copyright =C2=A9 2014 Taylan Ulrich Bay=C4=B1rl=C4=B1/Kammer It's 2015 now :) The same mistake is present in your other recent submissions. > +;;; > +;;; This file is part of GNU Guix. > +;;; > +;;; GNU Guix is free software; you can redistribute it and/or modify it > +;;; under the terms of the GNU General Public License as published by > +;;; the Free Software Foundation; either version 3 of the License, or (at > +;;; your option) any later version. > +;;; > +;;; GNU Guix is distributed in the hope that it will be useful, but > +;;; WITHOUT ANY WARRANTY; without even the implied warranty of > +;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > +;;; GNU General Public License for more details. > +;;; > +;;; You should have received a copy of the GNU General Public License > +;;; along with GNU Guix. If not, see . > + > +(define-module (gnu packages dnsmasq) > + #:use-module (guix licenses) > + #:use-module (guix packages) > + #:use-module (guix download) > + #:use-module (guix build-system gnu)) > + > +(define-public dnsmasq > + (package > + (name "dnsmasq") > + (version "2.72") > + (source (origin > + (method url-fetch) > + (uri (string-append > + "http://www.thekelleys.org.uk/dnsmasq/dnsmasq-" > + version ".tar.xz")) > + (sha256 > + (base32 > + "1c80hq09hfm8cp5pirfb8wdlc7dqkp7zzmbmdaradcvlblzx42vx"))= )) > + (build-system gnu-build-system) > + (arguments > + `(#:phases > + (alist-replace > + 'configure > + (lambda* (#:key inputs outputs #:allow-other-keys) > + (substitute* "Makefile" > + (("/usr/local") (assoc-ref outputs "out"))) > + (setenv "CC" "gcc")) > + %standard-phases) How about passing these as arguments to 'make' instead, like this? `(#:make-flags (list (string-append "PREFIX=3D" (assoc-ref %outputs "o= ut")) "CC=3Dgcc") #:phases (alist-delete 'configure %standard-phases) > + #:tests? #f)) Please add a comment explaining why this is needed. In this case "no check target" would be sufficient. > + (home-page "http://www.thekelleys.org.uk/dnsmasq/doc.html") > + (synopsis "Small caching DNS proxy and DHCP/TFTP server") > + (description > + "Dnsmasq is a lightweight DNS forwarder and DHCP server. It is des= igned > +to provide DNS and optionally, DHCP, to a small network. It can serve t= he > +names of local machines which are not in the global DNS. The DHCP server > +integrates with the DNS server and allows machines with DHCP-allocated > +addresses to appear in the DNS with names configured either in each host= or in > +a central configuration file. Dnsmasq supports static and dynamic DHCP = leases > +and BOOTP/TFTP for network booting of diskless machines.") > + (license gpl3))) The source files say that the user has the option of either GPLv2 or GPLv3. Therefore, please write (license gpl2 gpl3) with a comment to make it clear what this means. Can you post an updated patch? Thanks! Mark