From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Wurmus Subject: Re: Move r-{desolve, quadprog, pracma, subplex} from maths.scm to cran.scm Date: Fri, 22 Nov 2019 15:47:25 +0100 Message-ID: <871ru0c6iq.fsf@elephly.net> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:37077) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iYAHg-00063h-Ot for guix-devel@gnu.org; Fri, 22 Nov 2019 09:51:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iYADh-0006Km-6Q for guix-devel@gnu.org; Fri, 22 Nov 2019 09:47:38 -0500 In-reply-to: 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: zimoun Cc: Guix Devel Hi simon, > The packages r-desolve, r-quadprog, r-subplex and r-pracma are defined > in `maths.scm`. They come from the CRAN archive. For consistency, they > should be defined in the file `cran.scm`. Do you agree to move them? I think moving them to cran.scm is fine. In my opinion both of these places are similarly poor choices: maths.scm because we drag in the R build system and dependencies, and cran.scm because it=E2=80=99s huge and w= ill only get bigger. On the plus side, R code is rarely as =E2=80=9Cinvolved=E2=80=9D as Python = code, so I don=E2=80=99t expect cran.scm to become as tangled up with dependencies fro= m all over the place as python-xyz.scm, so letting cran.scm grow is probably just fine. Since nobody else seems to hold any strong opinions on this issue, I=E2=80= =99d say you=E2=80=99re welcome to move these package definitions to cran.scm. Please also update the Copyright comments at the top of the file where needed (use =E2=80=9Cgit blame=E2=80=9D and =E2=80=9Cgit log=E2=80=9D to si= mplify this task) and remove whatever spurious module imports there may be in maths.scm after removing the R package definitions. Thank you! -- Ricardo PS: I also think that CRAN things in bioinformatics.scm should be moved to cran.scm, and even some or all of the R stuff in statistics.scm. (Same applies to Bioconductor packages, which should end up in bioconductor.scm where possible.)