From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH] Add syslog.scm and rsyslog. Date: Mon, 28 Mar 2016 12:09:17 -0400 Message-ID: <20160328160917.GA7724@jasmine> References: <20160319182524.64f7f9b2@scratchpost.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:58880) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1akZjG-0003SI-LD for guix-devel@gnu.org; Mon, 28 Mar 2016 12:09:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1akZjC-0004q9-Js for guix-devel@gnu.org; Mon, 28 Mar 2016 12:09:22 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:48920) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1akZjC-0004q3-GH for guix-devel@gnu.org; Mon, 28 Mar 2016 12:09:18 -0400 Content-Disposition: inline In-Reply-To: <20160319182524.64f7f9b2@scratchpost.org> 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: Danny Milosavljevic Cc: guix-devel@gnu.org On Sat, Mar 19, 2016 at 06:25:24PM +0100, Danny Milosavljevic wrote: > Hi, > > this adds gnu/packages/syslog.scm and rsyslog. Hi, thanks for this, sorry for the delayed review! > If someone knowledgeable about which license text is which license can check the FIXMEs that would be nice. My approach to checking licenses is to do a web search for some phrase from the license, and then check the package's license text against the results of the search. Also, these are some good resources: https://www.gnu.org/licenses/license-list www.gnu.org/licenses/license-list.html > > That said, it seems inetutils also contains a syslogd (it also auto-starts; I didn't see it before) - so not sure how useful having another syslog is now. If rsyslog is a different program, then it's useful to someone! > --- > gnu/packages/syslog.scm | 110 ++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 110 insertions(+) > create mode 100644 gnu/packages/syslog.scm For your revisions, please submit each package as its own patch. You can read `git log` to learn the conventions for commit messages. Also, please remember to run `./pre-inst-env guix lint` on each package. Most of the changes that need to made to these packages will be described by `guix lint`. [...] > +;;; GNU Guix --- Functional package management for GNU > +;;; Copyright © 2013 Andreas Enge > +;;; Copyright © 2014, 2015 Mark H Weaver Don't forget to add a copyright line for yourself. > +(define-public libfastjson [...] > + (license license:gpl2+))) ; FIXME actually MIT. Which one? In the links I shared above, refer to the Expat and X11 licenses to see which "MIT" license this is. > +(define-public liblogging [...] > + (arguments `(#:configure-flags '("--disable-journal" ; to avoid libsystemd-journal > +))) If you leave the journal support enabled, does the program break when journald is absent? Users on systemd-based systems may want to use liblogging :) > +(define-public rsyslog [...] > + (inputs `(("libestr" ,libestr) > + ("libfastjson" ,libfastjson) > + ("zlib" ,zlib) > + ("util-linux" ,util-linux) Since util-linux is a "grab bag" of unrelated programs, I like to say in what it's being used for. There are some examples of this in gnu/packages. Can you submit a revised patch set?