From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] Emacs interface for Guix Date: Tue, 12 Aug 2014 16:19:34 +0200 Message-ID: <87egwlkcy1.fsf@gnu.org> References: <87k3719v7p.fsf@gmail.com> <87r419fa50.fsf@gnu.org> <87fvho9fqm.fsf@gmail.com> <87a97taixl.fsf@gmail.com> <87sil2rbly.fsf@gnu.org> <87tx5idn7f.fsf_-_@gmail.com> 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]:53687) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XHCvU-0004Af-AT for guix-devel@gnu.org; Tue, 12 Aug 2014 10:19:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XHCvJ-00071u-A5 for guix-devel@gnu.org; Tue, 12 Aug 2014 10:19:48 -0400 Received: from hera.aquilenet.fr ([2a01:474::1]:60022) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XHCvJ-00071F-3A for guix-devel@gnu.org; Tue, 12 Aug 2014 10:19:37 -0400 In-Reply-To: <87tx5idn7f.fsf_-_@gmail.com> (Alex Kost's message of "Tue, 12 Aug 2014 14:19:48 +0400") 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: Alex Kost Cc: guix-devel@gnu.org Alex Kost skribis: > Thanks for pointing. I've never contributed to a real project, so I > don't know the rules actually :) No problem. :-) There might still be unwritten rules, but we can fix that as we go. > From af4b8495969d70d59aa9f3f296628daeaf80b0d2 Mon Sep 17 00:00:00 2001 > From: Alex Kost > Date: Tue, 12 Aug 2014 12:32:16 +0400 > Subject: [PATCH 1/2] profiles: Add 'manifest-add'. > > * guix/profiles.scm (manifest-add): New procedure. > * tests/profiles.scm (guile-1.8.8): New variable. > ("manifest-add"): New test. Perfect. I=E2=80=99ve pushed it, followed by a patch that changes guix/scripts/package.scm to use =E2=80=98manifest-add=E2=80=99 (comments we= lcome.) > From 5fd45b3f4216921837f522d56b20c4be0a58fe8e Mon Sep 17 00:00:00 2001 > From: Alex Kost > Date: Tue, 12 Aug 2014 13:54:23 +0400 > Subject: [PATCH 2/2] guix package: Add 'process-package-actions'. > > * guix/scripts/package.scm (process-package-actions): New procedure. > (guix-package): Use it. > [ensure-default-profile]: Move to top-level. > [substitutes?]: New variable. > [same-package?]: Remove. > (options->installable, options->removable): Change according to > 'process-package-actions'. This patch would need to be rebased on top of f48624f. Were you planning on using =E2=80=98process-package-actions=E2=80=99 in the= Emacs interface? That seems like a coarse-grain and clumsy interface. Perhaps there are tinier parts of it that could be moved to (guix profiles)? For instance, there=E2=80=99s no =E2=80=98manifest-upgrade=E2=80=99 at the mome= nt. What about introducing a type that would contain a list of packages to install, to remove, and to upgrade, and we could do: ;; Show what will/would be installed, removed, etc. (show-transaction manifest transaction #:dry-run? bool) ;; Do the installation/removal/upgrades listed in TRANSACTION, and ;; return the new manifest. (manifest-perform-transaction manifest transaction) WDYT? Thanks, Ludo=E2=80=99.