From patchwork Sun Dec 22 18:23:03 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: "ashish.is--- via Guix-patches\" via" X-Patchwork-Id: 19427 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id B9C39178DD; Mon, 23 Dec 2019 12:44:15 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, TVD_RCVD_SPACE_BRACKET,T_DKIM_INVALID,UNPARSEABLE_RELAY,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 8E08717899 for ; Mon, 23 Dec 2019 12:44:14 +0000 (GMT) Received: from localhost ([::1]:56430 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ijN4H-0008Rj-Vo for patchwork@mira.cbaines.net; Mon, 23 Dec 2019 07:44:14 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:49069) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ijN49-0008Rd-DB for guix-patches@gnu.org; Mon, 23 Dec 2019 07:44:09 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ijN45-0003yu-Qd for guix-patches@gnu.org; Mon, 23 Dec 2019 07:44:05 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:44652) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ijN45-0003yk-Lv for guix-patches@gnu.org; Mon, 23 Dec 2019 07:44:01 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ijN45-0004rF-Iq for guix-patches@gnu.org; Mon, 23 Dec 2019 07:44:01 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38643] [PATCH] Add spacemacs package Resent-From: saffronsnail Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Mon, 23 Dec 2019 12:44:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 38643 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Ludovic =?utf-8?q?Court=C3=A8s?= Received: via spool by 38643-submit@debbugs.gnu.org id=B38643.157710502918644 (code B ref 38643); Mon, 23 Dec 2019 12:44:01 +0000 Received: (at 38643) by debbugs.gnu.org; 23 Dec 2019 12:43:49 +0000 Received: from localhost ([127.0.0.1]:50625 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ijN3o-0004qY-3A for submit@debbugs.gnu.org; Mon, 23 Dec 2019 07:43:49 -0500 Received: from eggs.gnu.org ([209.51.188.92]:39720) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ijN3i-0004qJ-Te for 38643@debbugs.gnu.org; Mon, 23 Dec 2019 07:43:42 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:55486) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ijN3d-0003cf-E6 for 38643@debbugs.gnu.org; Mon, 23 Dec 2019 07:43:33 -0500 Received: from lns-bzn-56-82-255-208-147.adsl.proxad.net ([82.255.208.147]:34714 helo=ribbon) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1ijN3K-0004D7-JS for 38643@debbugs.gnu.org; Mon, 23 Dec 2019 07:43:33 -0500 Resent-To: 38643@debbugs.gnu.org Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Resent-Date: Mon, 23 Dec 2019 13:43:11 +0100 Resent-Message-ID: <87woankyao.fsf@gnu.org> Received: from solo.fdn.fr ([unix socket]) by solo (Cyrus 2.5.10-Debian-2.5.10-3.2) with LMTPA; Sun, 22 Dec 2019 19:23:19 +0100 X-Sieve: CMU Sieve 2.4 Received: by solo.fdn.fr (Postfix) id 2F3CED09FE; Sun, 22 Dec 2019 19:23:19 +0100 (CET) Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by solo.fdn.fr (Postfix) with ESMTPS id E09ECD09F1 for ; Sun, 22 Dec 2019 19:23:18 +0100 (CET) Received: from fencepost.gnu.org ([2001:470:142:3::e]:46466) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ij5sr-0003WU-OR for ludovic.courtes@fdn.fr; Sun, 22 Dec 2019 13:23:17 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:54762) by fencepost.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1ij5sr-0000bZ-8E for ludo@gnu.org; Sun, 22 Dec 2019 13:23:17 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ij5sn-0003E4-Ag for ludo@gnu.org; Sun, 22 Dec 2019 13:23:17 -0500 Received: from mail1.protonmail.ch ([185.70.40.18]:19296) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ij5sm-0002xc-I4 for ludo@gnu.org; Sun, 22 Dec 2019 13:23:13 -0500 Date: Sun, 22 Dec 2019 18:23:03 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonmail.com; s=default; t=1577038986; bh=178pVWb4Lqd+5UcKGuA7vVF9IvDIWG86LMhfHzrqyBE=; h=Date:To:From:Reply-To:Subject:In-Reply-To:References:Feedback-ID: From; b=QrGz8vplPdeKTA/jb8aZAekhb/r3S1IdTHwWk8VIG4Y5VrfoZTHEfs1u5yH4GGNUH 7UQR3twY33HI6nAOFQmGo2XNWX1GN3AOWIclNEMWaTiWyfor4eSuS6NVWpdGpPiatc mQ+fgmTMQ8mPg7wZ4osJHqiZBiVpPmFtytwR9yUA= Message-ID: In-Reply-To: <87y2vbq8oi.fsf@gnu.org> References: <87y2vbq8oi.fsf@gnu.org> Feedback-ID: vsfXACN0M24HQHstP3yT1JWYyEzu1QcsRFwYKsjva9cdnZDX-Kw3LRJokFNAb7yQbpO_Kd2cGp4Y8Py4b4WW9Q==:Ext:ProtonMail MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: saffronsnail via Guix-patches From: "ashish.is--- via Guix-patches\" via" Reply-To: saffronsnail Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches > Do I get it right that Spacemacs will try to fetch the packages it needs > via pkg.el (ELPA)? > > Longer-term, it would be nice if it could fetch packages through Guix, > using Emacs-Guix. Are you familiar with this part of Spacemacs? I agree that this should be the long-term plan, though I would like to avoid the overhead of maintaining each individual ELPA package separately. I have not `guix import`ed an ELPA package before, but I know that for some other imports manual fixup is needed. Do you know if this is the case for ELPA? If so, I would prefer to work on a way to fix that than to manually maintain dozens of packages. To answer your question, I am have not worked much with this code but I had to poke around in it a bit to fix a bug. I know that Spacemacs has moved to using a stable (ie, maintained by the Spacemacs team) version of ELPA on the versioned release branch, so it might make sense to borrow some of the code form there. But I don't want to maintain a package for the versioned release because of the amount of time that has passed since the last release. In the long-long term, I'd like each Spacemacs layer to be a separate package that is independently installable (ideally without needing to clone the entire Spacemacs repository). In the shorter term, installing compiled versions of the files in the repsotiory instead of merely cloning it will be good, though I have not started looking at that yet so I am not sure how much effort that will take. > Please add the new files to gnu/local.mk (and mention that as well in > the commit log.) Done, though I'm not certian that the format of the changelog is correct. I modeled it after historical entries. It didn't look like the files in guix/build are supposed to be included here. > Please wrap lines to ~80 characters. :-) Each instance of these have been fixed. When I ran `guix lint` on my packages they warned me about long lines, but the threshold was above 80 characters - is this a problem with my invocation or a bug? > I believe this is incorrect ;-), could you adjust it? Copyright has been fixed in both places > Should be ‘gpl3+’ (meaning “version 3 or any later version”), if I’m not > mistaken. I do not believe this is correct. The `License` section of README.md in the root of the repository states 'GPLv3', without the '+'. Additionally, the header in `core/core-command-line.el` states `;;; License: GPLv3`. > We normally use a hyphen instead of underscore in the version string. Fixed > Please wrap lines as well. It seems difficult to format the builder in a way that is both readable and follows the line length standard, so I moved the builder code into the (guix build spacemacs-utils) module and stopped exporting the other symbols. Is this acceptable? A cursory search didn't find other packages that work this way. My concern is that the package is less readable when the builder is in a separate file, though this is not clearly worse than referring to helpers in a separate file. > Please add “#:use-module (gnu packages emacs)” add the top and refer to > ‘emacs’ directly, without the ‘@’ trick. Done! > As per , > please avoid abbreviations and add a docstring. How about calling it > ‘spacemacs-initialization-string’ or similar? Renamed to `create-initialization-code`, which I believe is the most accurate way to describe the procedure. Calling it a string made sense at first because the code is stored as a string, but I feel like that is a incedental attribute rather than an essential one. Let me know if you disagree. =) > For clarity, I’d recommend writing it as: > > (object->string > '(progn > (setq spacemacs-start-directory "\" spacemacs "/"") > …)) > > You can use ‘string-join’ and remove this procedure. It works like > this: > > --8<---------------cut here---------------start------------->8--- > scheme@(guile-user)> (string-join '("a" "b")) > $2 = "a b" > --8<---------------cut here---------------end--------------->8--- I was wanting procedures like these! Thank you! > Please add a docstring. Done! > Nitpick: I’d suggest writing it like this: > > (call-with-output-file output > (lambda (port) > (format port …))) I have no real preference either way, so I have changed it. I'm curious about your motivation though - is this just for consistency with the rest of the code base or is there a different practical/academic/aesthetic reason?. > Could you send an updated patch? Thank you! The updated patch is included below. Thanks, Bryan From d7185438daf8b2cd2e67c033a9a7079820bbafd2 Mon Sep 17 00:00:00 2001 From: Bryan Ferris Date: Wed, 11 Dec 2019 23:57:26 -0800 Subject: [PATCH] gnu: Add emacs-spacemacs * gnu/local.mk (GNU_SYSTEM_MODULES): add spacemacs.scm * gnu/local.mk (dist_patch_DATA): add spacemacs patches listed below * gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch: New file. * gnu/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch: New file. * gnu/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch: New file. * gnu/packages/spacemacs.scm: New file. * guix/build/spacemacs-utils.scm: New file. --- gnu/local.mk | 4 + ...acemacs-rolling-release-add-data-dir.patch | 259 ++++++++++++++++++ ...cs-rolling-release-inhibit-read-only.patch | 44 +++ ...s-rolling-release-quelpa-permissions.patch | 53 ++++ gnu/packages/spacemacs.scm | 119 ++++++++ guix/build/spacemacs-utils.scm | 72 +++++ 6 files changed, 551 insertions(+) create mode 100644 gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch create mode 100644 gnu/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch create mode 100644 gnu/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch create mode 100644 gnu/packages/spacemacs.scm create mode 100644 guix/build/spacemacs-utils.scm -- 2.24.0 diff --git a/gnu/local.mk b/gnu/local.mk index 0f62c8e1d7..1ed6dfc7ca 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -456,6 +456,7 @@ GNU_SYSTEM_MODULES = \ %D%/packages/slang.scm \ %D%/packages/smalltalk.scm \ %D%/packages/sml.scm \ + %D%/packages/spacemacs.scm \ %D%/packages/speech.scm \ %D%/packages/sphinx.scm \ %D%/packages/spice.scm \ @@ -1345,6 +1346,9 @@ dist_patch_DATA = \ %D%/packages/patches/scalapack-blacs-mpi-deprecations.patch \ %D%/packages/patches/scheme48-tests.patch \ %D%/packages/patches/scotch-build-parallelism.patch \ + %D%/packages/patches/spacemacs-rolling-release-add-data-dir.patch \ + %D%/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch \ + %D%/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch \ %D%/packages/patches/scotch-integer-declarations.patch \ %D%/packages/patches/sdl-libx11-1.6.patch \ %D%/packages/patches/sdl2-mesa-compat.patch \ diff --git a/gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch b/gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch new file mode 100644 index 0000000000..cb92d0b036 --- /dev/null +++ b/gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch @@ -0,0 +1,259 @@ +Spacemacs uses ~/.emacs.d in 2 ways: to store the code implementing the +spacemacs framework and to track state. When we tell spacemacs that it lives +in the store it tries to use the store location to track state as well. This +patch adds a new variable, spacemacs-data-directory, for keeping track of state. +This defaults to spacemacs-start-directory for upstream compatibility. + +diff --git a/core/core-configuration-layer.el b/core/core-configuration-layer.el +index 740b700cf..c8253dc9b 100644 +--- a/core/core-configuration-layer.el ++++ b/core/core-configuration-layer.el +@@ -120,7 +120,7 @@ subdirectory of ROOT is used." + "Hook executed at the end of configuration loading.") + + (defconst configuration-layer--elpa-root-directory +- (concat spacemacs-start-directory "elpa/") ++ (concat spacemacs-data-directory "elpa/") + "Spacemacs ELPA root directory.") + + (defconst configuration-layer--rollback-root-directory +diff --git a/core/core-documentation.el b/core/core-documentation.el +index 96bfc4a87..3a3ef456d 100644 +--- a/core/core-documentation.el ++++ b/core/core-documentation.el +@@ -289,7 +289,7 @@ preprocessors for the exported .org files." + }); + }); + ") +- (publish-target (concat spacemacs-start-directory "export/")) ++ (publish-target (concat spacemacs-data-directory "export/")) + (org-html-htmlize-output-type 'css) + (org-publish-project-alist + `(("spacemacs" +diff --git a/core/core-funcs.el b/core/core-funcs.el +index 28e7290a5..2f09c1df7 100644 +--- a/core/core-funcs.el ++++ b/core/core-funcs.el +@@ -274,7 +274,7 @@ result, incrementing passed-tests and total-tests." + (lambda (fname) + (when (file-exists-p fname) + (delete-file fname))) +- (directory-files-recursively user-emacs-directory "\\.elc$" t))) ++ (directory-files-recursively spacemacs-data-directory "\\.elc$" t))) + (byte-recompile-directory package-user-dir 0 arg)) + + (defun spacemacs/register-repl (feature repl-func &optional tag) +diff --git a/core/core-load-paths.el b/core/core-load-paths.el +index 718b27ce4..90c0c28ad 100644 +--- a/core/core-load-paths.el ++++ b/core/core-load-paths.el +@@ -19,6 +19,11 @@ + (defvar spacemacs-start-directory + user-emacs-directory + "Spacemacs start directory.") ++(defvar spacemacs-data-directory ++ user-emacs-directory ++ "Writable directory for spacemacs data") ++(unless (file-exists-p spacemacs-data-directory) ++ (make-directory spacemacs-data-directory)) + (defconst spacemacs-core-directory + (expand-file-name (concat spacemacs-start-directory "core/")) + "Spacemacs core directory.") +@@ -45,7 +50,7 @@ + (expand-file-name (concat spacemacs-banner-directory "img/heart.png")) + "Purple heart emoji.") + (defconst spacemacs-cache-directory +- (expand-file-name (concat user-emacs-directory ".cache/")) ++ (expand-file-name (concat spacemacs-data-directory ".cache/")) + "Spacemacs storage area for persistent files") + (defconst spacemacs-auto-save-directory + (expand-file-name (concat spacemacs-cache-directory "auto-save/")) +diff --git a/core/core-release-management.el b/core/core-release-management.el +index 989ccb124..db0f5c76d 100644 +--- a/core/core-release-management.el ++++ b/core/core-release-management.el +@@ -99,6 +99,10 @@ found." + (not (spacemacs//can-check-for-new-version-at-startup))) + (message (concat "Skipping check for new version " + "(reason: last check is too recent)"))) ++ ((and (not force) ++ (or (string-prefix-p "/gnu/store" spacemacs-start-directory) ++ (string-prefix-p "/nix/store" spacemacs-start-directory))) ++ (message (concat "Skipping check for new version (reason: on guix or nix)"))) + ((require 'async nil t) + (message "Start checking for new version...") + (async-start +diff --git a/core/libs/quelpa.el b/core/libs/quelpa.el +index 70b143657..9da420028 100644 +--- a/core/libs/quelpa.el ++++ b/core/libs/quelpa.el +@@ -79,7 +79,7 @@ the `:upgrade' argument." + :group 'quelpa + :type 'hook) + +-(defcustom quelpa-dir (expand-file-name "quelpa" user-emacs-directory) ++(defcustom quelpa-dir (expand-file-name "quelpa" spacemacs-data-directory) + "Where quelpa builds and stores packages." + :group 'quelpa + :type 'string) +diff --git a/tests/core/core-configuration-layer-utest.el b/tests/core/core-configuration-layer-utest.el +index 0d300142b..e5877932c 100644 +--- a/tests/core/core-configuration-layer-utest.el ++++ b/tests/core/core-configuration-layer-utest.el +@@ -880,16 +880,16 @@ + (ert-deftest test-make-layer--make-layer-from-symbol-with-a-dir () + (should (equal (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer +- 'layer nil nil spacemacs-start-directory)))) ++ 'layer nil nil spacemacs-data-directory)))) + + (ert-deftest test-make-layer--make-layer-from-spec-with-a-dir () + (should (equal (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer +- '(layer :name 'layer) nil nil spacemacs-start-directory)))) ++ '(layer :name 'layer) nil nil spacemacs-data-directory)))) + + (ert-deftest test-make-layer--cannot-make-layer-without-a-directory () + (mocker-let +@@ -908,10 +908,10 @@ + (ert-deftest test-make-layer--make-layer-without-a-dir-requires-an-initial-obj () + (let ((layer (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory))) ++ :dir spacemacs-data-directory))) + (should (equal (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer 'layer layer))))) + + ;; load packages +@@ -919,7 +919,7 @@ + (ert-deftest test-make-layer--make-used-layer-loads-packages-file () + (let ((layer (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-packages '(pkg1 pkg2 pkg3)) + (mocker-mock-default-record-cls 'mocker-stub-record)) + (mocker-let +@@ -931,13 +931,13 @@ + :variables nil + :packages '(pkg1 pkg2 pkg3) + :selected-packages 'all +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer 'layer layer 'used)))))) + + (ert-deftest test-make-layer--make-layer-force-load-packages-file-with-var () + (let ((layer (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-packages '(pkg1 pkg2 pkg3)) + (configuration-layer--load-packages-files t) + (mocker-mock-default-record-cls 'mocker-stub-record)) +@@ -950,13 +950,13 @@ + :variables nil + :packages '(pkg1 pkg2 pkg3) + :selected-packages 'all +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer 'layer layer)))))) + + (ert-deftest test-make-layer--make-layer-does-not-load-packages-file-by-default () + (let ((layer (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-packages '(pkg1)) + (mocker-mock-default-record-cls 'mocker-stub-record)) + (mocker-let +@@ -968,7 +968,7 @@ + (ert-deftest test-make-layer--make-used-layer-can-set-additional-properties () + (let ((layer (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-specs '(layer :disabled-for pkg8 pkg9 + :can-shadow layer2 layer3 + :variables foo bar toto 1)) +@@ -984,13 +984,13 @@ + :variables '(foo bar toto 1) + :packages '(pkg1 pkg2 pkg3) + :selected-packages 'all +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer layer-specs layer 'used)))))) + + (ert-deftest test-make-layer--make-not-used-layer-cannot-set-additional-properties () + (let ((layer (cfgl-layer "layer" + :name 'layer +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-specs '(layer :disabled-for pkg8 pkg9 + :can-shadow layer2 + :variables foo bar toto 1)) +@@ -1002,7 +1002,7 @@ + :variables nil + :packages nil + :selected-packages 'all +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer layer-specs layer))))) + + (ert-deftest test-make-layer--make-used-layer-can-override-additional-properties () +@@ -1011,7 +1011,7 @@ + :disabled-for '(pkg10) + :can-shadow '() + :variables '(titi tata tutu 1) +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-specs '(layer :disabled-for pkg8 pkg9 + :can-shadow layer2 + :variables foo bar toto 1)) +@@ -1027,7 +1027,7 @@ + :variables '(foo bar toto 1) + :packages '(pkg1 pkg2 pkg3) + :selected-packages 'all +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer layer-specs layer 'used)))))) + + (ert-deftest test-make-layer--make-not-used-layer-cannot-override-additional-properties () +@@ -1038,7 +1038,7 @@ + :variables '(titi tata tutu 1) + :packages '(pkg1 pkg2 pkg3) + :selected-packages 'all +- :dir spacemacs-start-directory)) ++ :dir spacemacs-data-directory)) + (layer-specs '(layer :disabled-for pkg8 pkg9 + :can-shadow '(layer2) + :variables foo bar toto 1)) +@@ -1050,7 +1050,7 @@ + :variables '(titi tata tutu 1) + :packages '(pkg1 pkg2 pkg3) + :selected-packages 'all +- :dir spacemacs-start-directory) ++ :dir spacemacs-data-directory) + (configuration-layer/make-layer layer-specs layer))))) + + ;; --------------------------------------------------------------------------- +diff --git a/tests/doc/org-export-ftest.el b/tests/doc/org-export-ftest.el +index 4ee86c897..cdd731eec 100644 +--- a/tests/doc/org-export-ftest.el ++++ b/tests/doc/org-export-ftest.el +@@ -59,6 +59,6 @@ + ;; ----------------------------------------------------------------------------- + (ert-deftest test-spacemacs-html-export () + (unwind-protect (spacemacs/publish-doc) +- (delete-directory (concat spacemacs-start-directory ++ (delete-directory (concat spacemacs-data-directory + "export/") + t))) diff --git a/gnu/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch b/gnu/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch new file mode 100644 index 0000000000..4cb184a898 --- /dev/null +++ b/gnu/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch @@ -0,0 +1,44 @@ +The first time that a user runs spacemacs it will open the default +configuration from the source tree and modify the buffer. The buffer is +read-only since the file it opened is, but since we save the file to the users +home directory and not back to it's original location temporarily inhibiting +read-only solves our problem. + +diff --git a/core/core-dotspacemacs.el b/core/core-dotspacemacs.el +index f28249148..3b7872ff0 100644 +--- a/core/core-dotspacemacs.el ++++ b/core/core-dotspacemacs.el +@@ -643,19 +643,20 @@ If ARG is non nil then Ask questions to the user before installing the dotfile." + (with-current-buffer (find-file-noselect + (concat dotspacemacs-template-directory + ".spacemacs.template")) +- (dolist (p preferences) +- (goto-char (point-min)) +- (re-search-forward (car p)) +- (replace-match (cadr p))) +- (let ((install +- (if (file-exists-p dotspacemacs-filepath) +- (y-or-n-p +- (format "%s already exists. Do you want to overwrite it ? " +- dotspacemacs-filepath)) t))) +- (when install +- (write-file dotspacemacs-filepath) +- (message "%s has been installed." dotspacemacs-filepath) +- t)))) ++ (let ((inhibit-read-only t)) ++ (dolist (p preferences) ++ (goto-char (point-min)) ++ (re-search-forward (car p)) ++ (replace-match (cadr p))) ++ (let ((install ++ (if (file-exists-p dotspacemacs-filepath) ++ (y-or-n-p ++ (format "%s already exists. Do you want to overwrite it ? " ++ dotspacemacs-filepath)) t))) ++ (when install ++ (write-file dotspacemacs-filepath) ++ (message "%s has been installed." dotspacemacs-filepath) ++ t))))) + (dotspacemacs/load-file) + ;; force new wizard values to be applied + (dotspacemacs/init)) diff --git a/gnu/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch b/gnu/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch new file mode 100644 index 0000000000..96016d6935 --- /dev/null +++ b/gnu/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch @@ -0,0 +1,53 @@ +When quelpa builds some packages it has to copy files from the store, which +means that they are marked as read-only. This patch sets the permissions of +copied files and directories to ensure that they are writable. + +diff --git a/core/libs/quelpa.el b/core/libs/quelpa.el +index 70b143657..9da420028 100644 +@@ -374,8 +374,10 @@ and return TIME-STAMP, otherwise return OLD-TIME-STAMP." + (delete-directory dir t) + (make-directory dir) + (if (eq type 'file) +- (copy-file file-path dir t t t t) +- (copy-directory file-path dir t t t))) ++ (progn (copy-file file-path dir t t t t) ++ (set-file-modes (expand-file-name (file-name-nondirectory file-path) dir) #o644)) ++ (progn (copy-directory file-path dir t t t) ++ (set-file-modes dir #o755)))) + (quelpa-build--dump new-stamp-info stamp-file) + (quelpa-file-version file-path type version time-stamp)))) + +@@ -1369,10 +1371,13 @@ FILES is a list of (SOURCE . DEST) relative filepath pairs." + (cond + ((file-regular-p file) + (quelpa-build--message "%s -> %s" file newname) +- (copy-file file newname)) ++ (copy-file file newname) ++ (set-file-modes newname #o644)) + ((file-directory-p file) + (quelpa-build--message "%s => %s" file newname) +- (copy-directory file newname)))) ++ (make-directory newname) ++ (set-file-modes newname #o755) ++ (map quelpa-build--copy-file (directory-files file t))))) + + (defun quelpa-build--find-source-file (target files) + "Search for source of TARGET in FILES." +@@ -1463,6 +1468,7 @@ Returns the archive entry for the package." + (if (file-exists-p pkg-target) + (quelpa-build--message "Skipping rebuild of %s" pkg-target) + (copy-file pkg-source pkg-target) ++ (set-file-modes pkg-target #o644) + (let ((enable-local-variables nil) + (make-backup-files nil)) + (with-temp-buffer +@@ -1566,7 +1572,8 @@ attribute with an URL like \"http://domain.tld/path/to/file.el\"." + (unless (string= (file-name-extension url) "el") + (error "<%s> does not end in .el" url)) + (unless (file-directory-p dir) +- (make-directory dir)) ++ (make-directory dir) ++ (set-file-modes dir #o755)) + (url-copy-file url local-path t) + (quelpa-check-hash name config local-path dir 'url))) + diff --git a/gnu/packages/spacemacs.scm b/gnu/packages/spacemacs.scm new file mode 100644 index 0000000000..24b3f23428 --- /dev/null +++ b/gnu/packages/spacemacs.scm @@ -0,0 +1,119 @@ +;;; GNU Guix --- Functional package management for GNU +;;; Copyright © 2019 Bryan Ferris +;;; +;;; 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 spacemacs) + #:use-module (gnu packages) + #:use-module (gnu packages base) + #:use-module (gnu packages bash) + #:use-module (gnu packages compression) + #:use-module (gnu packages emacs) + + #:use-module (guix build-system trivial) + #:use-module (guix download) + #:use-module (guix gexp) + #:use-module (guix git-download) + #:use-module ((guix licenses) #:prefix license:) + #:use-module (guix packages) + #:use-module (guix utils)) + +(define-public spacemacs-rolling-release + (let ((commit "1e278a3cb9cd4730ee17416b55fb778b62da2fd0")) + (package + (name "spacemacs-rolling-release") + (version (git-version "0.3.0" "0" commit)) + (source (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/syl20bnr/spacemacs") + (commit commit))) + (sha256 + (base32 + "17yxgchj7qilgljpjai3ad0pzj7k6sq6gqbnxrvcizvkvcnv10z5")) + (file-name (string-append name "-" version)) + (patches + (search-patches + "spacemacs-rolling-release-add-data-dir.patch" + "spacemacs-rolling-release-inhibit-read-only.patch" + "spacemacs-rolling-release-quelpa-permissions.patch")))) + (build-system trivial-build-system) + (native-inputs `(("tar" ,tar) ("xz" ,xz))) + (arguments (list + #:modules '((guix build utils)) + #:builder '(begin (use-modules (guix build utils)) + (setenv "PATH" + (string-append (getenv "PATH") ":" + (assoc-ref + %build-inputs "xz") + "/bin")) + (mkdir-p (assoc-ref %outputs "out")) + (system* (string-append + (assoc-ref %build-inputs "tar") + "/bin/tar") + "xf" (assoc-ref %build-inputs + "source") + "-C" (assoc-ref %outputs "out") + "--strip-components" "1")))) + (synopsis "Automatically configured emacs for both emacs and vim users") + (description "Spacemacs is a configuration framework for emacs designed to +work well for people with experience using either emacs or vim. It has 4 +driving principles: mnemonics, discoverability, consistency, and crowd +configuration. Mnemonics mean that key bindings use letters that stand for the +action they take, making the easier to remember. Discoverability means that +help is displayed when partial keybindings are entered, and prepared +configuration units are searchable. Consistency means that bindings for +different use-cases (eg, different programming languages) use the same +keybindings for similar actions. And crowd-configuration means that the +spacemacs community collaborates to provide the best default experience for new +and expert users alike.") + (home-page "https://spacemacs.org") + (license license:gpl3)))) + +(define* (generate-wrapped-emacs-spacemacs emacs spacemacs + #:optional (name "emacs-spacemacs")) + "Given an emacs package and a spacemacs package, create wrappers that allow +the use of spacemacs without conflicting with the base emacs." + (package + (name name) + (version (string-append (package-version emacs) "-" + (package-version spacemacs))) + (source #f) + (build-system trivial-build-system) + (inputs `(("sh" ,bash) + ("emacs" ,emacs) + ("spacemacs" ,spacemacs))) + (arguments `(#:modules ((guix build utils) (guix build spacemacs-utils)) + #:builder (begin + (use-modules ((guix build spacemacs-utils) + #:prefix spacemacs:)) + (spacemacs:builder + #:shell (string-append + (assoc-ref %build-inputs "sh") + "/bin/sh") + #:emacs (string-append + (assoc-ref %build-inputs "emacs") + "/bin/emacs") + #:spacemacs (assoc-ref %build-inputs "spacemacs") + #:out (string-append + (assoc-ref %outputs "out") "/bin"))))) + (home-page (package-home-page spacemacs)) + (synopsis (package-synopsis spacemacs)) + (description (package-description spacemacs)) + (license (package-license spacemacs)))) + +(define-public emacs-spacemacs + (generate-wrapped-emacs-spacemacs emacs spacemacs-rolling-release)) diff --git a/guix/build/spacemacs-utils.scm b/guix/build/spacemacs-utils.scm new file mode 100644 index 0000000000..5124c41263 --- /dev/null +++ b/guix/build/spacemacs-utils.scm @@ -0,0 +1,72 @@ +;;; GNU Guix --- Functional package management for GNU +;;; Copyright © 2019 Bryan Ferris +;;; +;;; 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 (guix build spacemacs-utils) + #:use-module (guix build utils) + #:use-module (oop goops) + #:use-module (srfi srfi-1) + + #:export (builder)) + +(define (create-initialization-code spacemacs) + "Create elisp code that sets spacemacs-specific variables and then loads the +spacemacs initialization file" + (object->string + `(progn + (setq spacemacs-start-directory (concat ,spacemacs "/")) + (setq spacemacs-data-directory + (concat (or (getenv "XDG_DATA_DIR") + (concat (getenv "HOME") "/.local/share")) + "/spacemacs/")) + (setq package-user-dir (concat spacemacs-data-directory "elpa/")) + (load-file (concat spacemacs-start-directory "init.el"))))) + +(define (generate-wrapper shell output executable . args) + "create a shell script interpreted by sh-compatible shell `shell` that +executes `executable` passing arguments `args` as well as any passed in from +the command line." + (call-with-output-file + output (lambda (port) + (format port "~A~%~A" + (string-append "#!" shell) + (string-join (list "exec" "-a" shell + executable (string-join args) + "\"$@\""))))) + (chmod output #o555)) + +(define* (builder #:key shell emacs spacemacs out) + "Create exectables that run emacs, the emacs server, and the emacs client +with Spacemacs code preloaded." + (mkdir-p out) + + (generate-wrapper shell + (string-append out "/spacemacs") + emacs " --no-init-file" "--eval" + (string-append "'" + (create-initialization-code spacemacs) + "'")) + + (generate-wrapper shell + (string-append out "/spacemacsdaemon") + (string-append out "/spacemacs") + "--daemon=spacemacs") + + (generate-wrapper shell + (string-append out "/spacemacsclient") + (string-append emacs "client") + "--socket-name" "spacemacs"))