[PATCH] Add spacemacs package

  • Done
  • quality assurance status badge
Details
3 participants
  • Ludovic Courtès
  • saffronsnail
  • zimoun
Owner
unassigned
Submitted by
saffronsnail
Severity
normal
S
S
saffronsnail wrote on 17 Dec 2019 00:05
(name . guix-patches@gnu.org)(address . guix-patches@gnu.org)
mS425o3U--g_ZZemWqvuUgtTHvBroyNZvJUCP6Dy2ABWdMTmgI1CweiSpOj40xlg1LXarBqJE0krzRh4J-DhzoWQ_jofFDDgxUXg1cvjZUA=@protonmail.com
Hello,

I have been using a local package for installing Spacemacs (https://github.com/syl20bnr/spacemacs)and would like to contribute it. I ran through the packaging guidelines and believe that everything should be in order - though note that the way I personally format lisp code is not standard, so while I tried to match the style I saw in the repository there may be some feedback there. There are 2 new packages added, spacemacs-rolling-release (to add the spacemacs code to the store) and emacs-spacemacs (to install the spacemacs command which launches vanilla emacs with command-line options to load emacs, which allows for side-by-side installations of vanilla emacs and spacemacs). There are 3 patch files which have been included to address bugs that arise when upstream spacemacs is installed to a read-only filesystem.

Both packages have 2 lint warnings which I believe are acceptable. Each has a lint error for the description line being too long, but the description follows the advice in the "Synopses and Descriptions" section of the manual, and I can't break it into multiple lines because it must be a string literal. The other warning is that the TLS certificate of spacemacs.org, the package's main website, does not match that of github.com, where the code is hosted.

Below is the output of `guix format-patch -1`. Let me know if any additional work is needed.

Thanks,
Bryan

From 67e44b3cdec7248d00ca2b10b5617738ab3f0d45 Mon Sep 17 00:00:00 2001
From: Bryan Ferris <saffronsnail@protonmail.com>
Date: Wed, 11 Dec 2019 23:57:26 -0800
Subject: [PATCH] gnu: Add emacs-spacemacs

* 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.
---
...acemacs-rolling-release-add-data-dir.patch | 255 ++++++++++++++++++
...cs-rolling-release-inhibit-read-only.patch | 40 +++
...s-rolling-release-quelpa-permissions.patch | 51 ++++
gnu/packages/spacemacs.scm | 117 ++++++++
guix/build/spacemacs-utils.scm | 47 ++++
5 files changed, 510 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

Toggle diff (348 lines)
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..74b35aec97
--- /dev/null
+++ b/gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch
@@ -0,0 +1,255 @@
+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."
+ });
+ });
+ </script>")
+- (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..71d3858696
--- /dev/null
+++ b/gnu/packages/patches/spacemacs-rolling-release-inhibit-read-only.patch
@@ -0,0 +1,40 @@
+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..b3b514941f
--- /dev/null
+++ b/gnu/packages/patches/spacemacs-rolling-release-quelpa-permissions.patch
@@ -0,0 +1,51 @@
+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-
This message was truncated. Download the full message here.
Attachment: file
L
L
Ludovic Courtès wrote on 17 Dec 2019 16:21
(name . saffronsnail)(address . saffronsnail@protonmail.com)(address . 38643@debbugs.gnu.org)
87y2vbq8oi.fsf@gnu.org
Hello!

saffronsnail <saffronsnail@protonmail.com> skribis:

Toggle quote (2 lines)
> I have been using a local package for installing Spacemacs (https://github.com/syl20bnr/spacemacs)and would like to contribute it. I ran through the packaging guidelines and believe that everything should be in order - though note that the way I personally format lisp code is not standard, so while I tried to match the style I saw in the repository there may be some feedback there. There are 2 new packages added, spacemacs-rolling-release (to add the spacemacs code to the store) and emacs-spacemacs (to install the spacemacs command which launches vanilla emacs with command-line options to load emacs, which allows for side-by-side installations of vanilla emacs and spacemacs). There are 3 patch files which have been included to address bugs that arise when upstream spacemacs is installed to a read-only filesystem.

Awesome!

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?

Some (minor) comments follow:

Toggle quote (11 lines)
> From 67e44b3cdec7248d00ca2b10b5617738ab3f0d45 Mon Sep 17 00:00:00 2001
> From: Bryan Ferris <saffronsnail@protonmail.com>
> Date: Wed, 11 Dec 2019 23:57:26 -0800
> Subject: [PATCH] gnu: Add emacs-spacemacs
>
> * 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.

Please add the new files to gnu/local.mk (and mention that as well in
the commit log.)

Toggle quote (5 lines)
> --- /dev/null
> +++ b/gnu/packages/patches/spacemacs-rolling-release-add-data-dir.patch
> @@ -0,0 +1,255 @@
> +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.

Please wrap lines to ~80 characters. :-)

Toggle quote (5 lines)
> +++ b/gnu/packages/spacemacs.scm
> @@ -0,0 +1,117 @@
> +;;; GNU Guix --- Functional package management for GNU
> +;;; Copyright © 2016 David Craven <david@craven.ch>

I believe this is incorrect ;-), could you adjust it?

Toggle quote (34 lines)
> +(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.")

Please wrap lines to 80 characters as well (you can look at the other
files for style hints.)

Toggle quote (3 lines)
> + (home-page "https://spacemacs.org")
> + (license license:gpl3))))

Should be ‘gpl3+’ (meaning “version 3 or any later version”), if I’m not
mistaken.

Toggle quote (6 lines)
> +(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)))

We normally use a hyphen instead of underscore in the version string.

Toggle quote (38 lines)
> + (source #f)
> + (build-system trivial-build-system)
> + (inputs `(("sh" ,bash)
> + ("emacs" ,emacs)
> + ("spacemacs" ,spacemacs)))
> + (arguments `(#:modules ((guix build spacemacs-utils) (guix build utils) (srfi srfi-1))
> + #:builder (begin (use-modules (guix build spacemacs-utils) (guix build utils) (srfi srfi-1))
> + (let* ((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"))
> +
> + (init-code (create-init-string spacemacs)))
> + (mkdir-p out)
> +
> + (generate-wrapper shell
> + (string-append out "/spacemacs")
> + emacs " --no-init-file" "--eval"
> + init-code)
> +
> + (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")))))

Please wrap lines as well.

Toggle quote (3 lines)
> +(define-public emacs-spacemacs
> + (generate-wrapped-emacs-spacemacs (@ (gnu packages emacs) emacs) spacemacs-rolling-release))

Please add “#:use-module (gnu packages emacs)” add the top and refer to
‘emacs’ directly, without the ‘@’ trick.

Toggle quote (5 lines)
> +++ b/guix/build/spacemacs-utils.scm
> @@ -0,0 +1,47 @@
> +;;; GNU Guix --- Functional package management for GNU
> +;;; Copyright © 2016 David Craven <david@craven.ch>

That one also. :-)

Toggle quote (2 lines)
> +(define (create-init-string spacemacs)

please avoid abbreviations and add a docstring. How about calling it
‘spacemacs-initialization-string’ or similar?

Toggle quote (7 lines)
> + (string-append "\"(progn (setq spacemacs-start-directory \\\"" 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\\\")))\""))

For clarity, I’d recommend writing it as:

(object->string
'(progn
(setq spacemacs-start-directory "\" spacemacs "/"")
…))

Toggle quote (6 lines)
> +(define (string-append-with-space arg . arglist)
> + (if (nil? arglist)
> + arg
> + (apply string-append-with-space
> + (string-append arg " " (first arglist)) (drop arglist 1))))

You can use ‘string-join’ and remove this procedure. It works like
this:

Toggle snippet (4 lines)
scheme@(guile-user)> (string-join '("a" "b"))
$2 = "a b"

Toggle quote (2 lines)
> +(define (generate-wrapper shell output executable . args)

Please add a docstring.

Toggle quote (8 lines)
> + (with-output-to-file
> + output (lambda ()
> + (format #t (string-append "#!" shell "~%"
> + (apply string-append-with-space
> + "exec" "-a" shell executable args)
> + " \"$@\""))))
> + (chmod output #o555))

Nitpick: I’d suggest writing it like this:

(call-with-output-file output
(lambda (port)
(format port …)))

Could you send an updated patch? Thank you!

Ludo’.
S
S
saffronsnail wrote on 22 Dec 2019 19:23
(name . Ludovic Courtès)(address . ludo@gnu.org)
X7kuwz6we7bMgo9FoBKkSRozGy8kO1DQPbYLA2OvWJ2_G2a_yS11G2Oq3cjhwmxLHg4RLC4ugs599vI_oYNnOluorFJ0V_nC6xJcPoufNr4=@protonmail.com
Toggle quote (6 lines)
> 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.

Toggle quote (3 lines)
> 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.

Toggle quote (2 lines)
> 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?

Toggle quote (2 lines)
> I believe this is incorrect ;-), could you adjust it?

Copyright has been fixed in both places

Toggle quote (3 lines)
> 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`.

Toggle quote (2 lines)
> We normally use a hyphen instead of underscore in the version string.

Fixed

Toggle quote (2 lines)
> 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.

Toggle quote (3 lines)
> Please add “#:use-module (gnu packages emacs)” add the top and refer to
> ‘emacs’ directly, without the ‘@’ trick.

Done!

Toggle quote (4 lines)
> 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. =)

Toggle quote (15 lines)
> 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!

Toggle quote (2 lines)
> Please add a docstring.

Done!

Toggle quote (6 lines)
> 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?.

Toggle quote (2 lines)
> 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 <saffronsnail@protonmail.com>
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

Toggle diff (287 lines)
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."
+ });
+ });
+ </script>")
+- (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/")
+
This message was truncated. Download the full message here.
Z
Z
zimoun wrote on 12 Apr 2022 12:36
Re: bug#38643: [PATCH] Add spacemacs package
(name . saffronsnail)(address . saffronsnail@protonmail.com)
867d7u37xg.fsf_-_@gmail.com
Hi,

This patch #38643 submission had fallen into the cracks



What is the next action? Is it worth to include it despite the fact
that:

> 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.

or do we drop it?



Cheers,
simon
L
L
Ludovic Courtès wrote on 12 Apr 2022 20:12
(name . zimoun)(address . zimon.toutoune@gmail.com)
87o816cgrg.fsf@gnu.org
Hi,

zimoun <zimon.toutoune@gmail.com> skribis:

Toggle quote (20 lines)
> What is the next action? Is it worth to include it despite the fact
> that:
>
> > 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.
>
> or do we drop it?

Given that it hasn’t seen any activity, I believe we can close it for
now. I’d love to see proper Spacemacs integration, but for that we need
saffronsnail or another Spacemacs-savvy person to champion it.

Thanks,
Ludo’.
Closed
?