From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:55748) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hEEwL-0002Bq-1r for guix-patches@gnu.org; Wed, 10 Apr 2019 11:15:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hEEoZ-0007en-5Y for guix-patches@gnu.org; Wed, 10 Apr 2019 11:07:05 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:39895) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hEEoY-0007du-3Z for guix-patches@gnu.org; Wed, 10 Apr 2019 11:07:03 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hEEoX-0002Vk-P6 for guix-patches@gnu.org; Wed, 10 Apr 2019 11:07:01 -0400 Subject: [bug#35086] [PATCH] New package file for BOINC client & server Resent-Message-ID: From: Ludovic =?UTF-8?Q?Court=C3=A8s?= References: <17ee4c77-7bf9-4a4c-b856-f9ca4ff3c7db@www.fastmail.com> Date: Wed, 10 Apr 2019 17:06:06 +0200 In-Reply-To: <17ee4c77-7bf9-4a4c-b856-f9ca4ff3c7db@www.fastmail.com> (Brant Gardner's message of "Tue, 02 Apr 2019 12:24:01 -0400") Message-ID: <87zhoxc335.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 8bit List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Brant Gardner Cc: 35086@debbugs.gnu.org Hello Brant, "Brant Gardner" skribis: > This is my first submission for contributing to Guix, so any feedback or advice is welcomed! Thanks for joining! I have some suggestions below, but overall its a great start! > From 2946bd1c83a2ea224841d22463992be599dfc385 Mon Sep 17 00:00:00 2001 > From: Brant Gardner > Date: Tue, 2 Apr 2019 11:09:45 -0500 > Subject: [PATCH] Added distributed.scm > > New package file for BOINC and other distributed-computing projects. This > initial file just includes boinc-client and boinc-server, both at version > 7.14.2. The convention is to write commit logs in ChangeLog format, as described at: https://www.gnu.org/prep/standards/html_node/Change-Logs.html But you can simply look at git log and mimic what you see. (You might also want to take a look at if you havent already.) As for the module name, what about boinc.scm instead? Distributed sounds a bit too generic to me. > +++ b/gnu/packages/distributed.scm > @@ -0,0 +1,119 @@ > +;;; GNU Guix --- Functional package management for GNU > +;;; Copyright .© 2014, 2015 Ludovic Court.¨s > +;;; Copyright .© 2016, 2017, 2018, 2019 Efraim Flashner > +;;; Copyright .© 2018 Tobias Geerinckx-Rice > +;;; Copyright .© 2018 Mathieu Othacehe It should be just you here. :-) > + #:use-module (gnu packages autotools) > + #:use-module (gnu packages base) > + #:use-module (gnu packages compression) > + #:use-module (gnu packages databases) > + #:use-module (gnu packages pkg-config) > + #:use-module (gnu packages tls) > + #:use-module (gnu packages curl) > + #:use-module (gnu packages wxwidgets) > + #:use-module (gnu packages gnome) > + #:use-module (gnu packages gtk) > + #:use-module (gnu packages perl) > + #:use-module (gnu packages sqlite) > + #:use-module (gnu packages python) > + #:use-module (gnu packages python-xyz)) I think you can trim this list. > + (synopsis "BOINC lets you help cutting-edge science research using your computer") What about something like Contribute computing power to research projects? > + (description "BOINC is a platform for high-throughput computing on a large > +scale (thousands or millions of computers). It can be used for volunteer > +computing (using consumer devices) or grid computing (using organizational > +resources). It supports virtualized, parallel, and GPU-based applications. > + > +BOINC is distributed under the LGPL open source license. It can be used for > +commercial purposes, and applications need not be open source.") This last paragraph is unnecessary: everything in Guix is free software, so we never mention it in descriptions. > + (synopsis "BOINC lets you help cutting-edge science research using your computer") > + (description "BOINC is a platform for high-throughput computing on a large > +scale (thousands or millions of computers). It can be used for volunteer > +computing (using consumer devices) or grid computing (using organizational > +resources). It supports virtualized, parallel, and GPU-based applications. It would be nice to have a slightly different synopsis and description for the server and for the client. Apart from that it LGTM! Could you double check that guix lint is happy and send an updated patch? Thank you! Ludo.