[bug#55419] [PATCH 1/5] gnu: emacs-next-pgtk: Use new package style.

[bug#55419] [PATCH 1/5] gnu: emacs-next-pgtk: Use new package style.

[Liliana Marie Prikler]

* gnu/packages/emacs.scm (emacs-next-pgtk)[arguments]: Convert to list of
G-Expressions.
[inputs]: Use modify-inputs.
---
 gnu/packages/emacs.scm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 9649334e45..af902ac9f9 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -348,13 +348,13 @@ (define-public emacs-next-pgtk
     (name "emacs-next-pgtk")
     (arguments
      (substitute-keyword-arguments (package-arguments emacs-next)
-       ((#:configure-flags flags ''())
-        `(cons* "--with-pgtk" "--with-xwidgets" ,flags))))
+       ((#:configure-flags flags #~'())
+        #~(cons* "--with-pgtk" "--with-xwidgets" #$flags))))
     (propagated-inputs
      (list gsettings-desktop-schemas glib-networking))
     (inputs
-     `(("webkitgtk" ,webkitgtk-with-libsoup2)
-       ,@(package-inputs emacs-next)))
+     (modify-inputs (package-inputs emacs-next)
+       (prepend webkitgtk-with-libsoup2)))
     (home-page "https://github.com/masm11/emacs")
     (synopsis "Emacs text editor with @code{pgtk} and @code{xwidgets} support")
     (description "This Emacs build implements graphical UI purely in terms of
-- 
2.36.0

[bug#55419] [PATCH 2/5] gnu: emacs-xwidgets: Use new package style.

[Liliana Marie Prikler]

* gnu/packages/emacs.scm (emacs-xwidgets)[arguments]: Convert to list of
G-Expressions.
[inputs]: Use modify-inputs.
---
 gnu/packages/emacs.scm | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index af902ac9f9..e3eba851ff 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -391,16 +391,15 @@ (define-public emacs-xwidgets
     (build-system gnu-build-system)
     (arguments
      (substitute-keyword-arguments (package-arguments emacs)
-       ((#:configure-flags flags ''())
-        `(cons "--with-xwidgets" ,flags))
+       ((#:configure-flags flags #~'())
+        #~(cons "--with-xwidgets" #$flags))
        ((#:phases phases)
-        `(modify-phases ,phases
-           (delete 'restore-emacs-pdmp)
-           (delete 'strip-double-wrap)))))
+        #~(modify-phases #$phases
+            (delete 'restore-emacs-pdmp)
+            (delete 'strip-double-wrap)))))
     (inputs
-     `(("webkitgtk" ,webkitgtk-with-libsoup2)
-       ("libxcomposite" ,libxcomposite)
-       ,@(package-inputs emacs)))))
+     (modify-inputs (package-inputs emacs)
+       (prepend webkitgtk-with-libsoup2 libxcomposite)))))
 
 (define-public emacs-no-x
   (package/inherit emacs
-- 
2.36.0

[bug#55419] [PATCH 3/5] gnu: emacs-no-x: Use new package style.

[Liliana Marie Prikler]

* gnu/packages/emacs.scm (emacs-no-x)[arguments]: Convert to list of
G-Expressions.
[inputs]: Use modify-inputs.
---
 gnu/packages/emacs.scm | 24 +++++++++++-------------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index e3eba851ff..027afc811a 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -407,22 +407,20 @@ (define-public emacs-no-x
     (synopsis "The extensible, customizable, self-documenting text
 editor (console only)")
     (build-system gnu-build-system)
-    (inputs (fold alist-delete
-                  (package-inputs emacs)
-                  '("libx11" "gtk+" "libxft" "libtiff" "giflib" "libjpeg"
-                    "imagemagick" "libpng" "librsvg" "libxpm" "libice"
-                    "libsm" "cairo" "pango" "harfbuzz"
-
-                    ;; These depend on libx11, so remove them as well.
-                    "libotf" "m17n-lib" "dbus")))
+    (inputs (modify-inputs (package-inputs emacs)
+              (delete "libx11" "gtk+" "libxft" "libtiff" "giflib" "libjpeg"
+                      "imagemagick" "libpng" "librsvg" "libxpm" "libice"
+                      "libsm" "cairo" "pango" "harfbuzz"
+                      ;; These depend on libx11, so remove them as well.
+                      "libotf" "m17n-lib" "dbus")))
     (arguments
      (substitute-keyword-arguments (package-arguments emacs)
-       ((#:configure-flags flags ''())
-        `(delete "--with-cairo" ,flags))
+       ((#:configure-flags flags #~'())
+        #~(delete "--with-cairo" #$flags))
        ((#:phases phases)
-        `(modify-phases ,phases
-           (delete 'restore-emacs-pdmp)
-           (delete 'strip-double-wrap)))))))
+        #~(modify-phases #$phases
+            (delete 'restore-emacs-pdmp)
+            (delete 'strip-double-wrap)))))))
 
 (define-public emacs-no-x-toolkit
   (package/inherit emacs
-- 
2.36.0

[bug#55419] [PATCH 4/5] gnu: emacs-no-x-toolkit: Use new package style.

[Liliana Marie Prikler]

* gnu/packages/emacs.scm (emacs-no-x-toolkit)[arguments]: Convert to list of
G-Expressions.
[inputs]: Use modify-inputs.
---
 gnu/packages/emacs.scm | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 027afc811a..392b3297b0 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -428,14 +428,15 @@ (define-public emacs-no-x-toolkit
     (synopsis "The extensible, customizable, self-documenting text
 editor (without an X toolkit)" )
     (build-system gnu-build-system)
-    (inputs (append `(("inotify-tools" ,inotify-tools))
-                    (alist-delete "gtk+" (package-inputs emacs))))
+    (inputs (modify-inputs (package-inputs emacs)
+              (delete "gtk+")
+              (prepend inotify-tools)))
     (arguments
      (substitute-keyword-arguments (package-arguments emacs)
-       ((#:configure-flags flags ''())
-        `(cons "--with-x-toolkit=no" ,flags))
+       ((#:configure-flags flags #~'())
+        #~(cons "--with-x-toolkit=no" #$flags))
        ((#:phases phases)
-        `(modify-phases ,phases
+        #~(modify-phases #$phases
            (delete 'restore-emacs-pdmp)
            (delete 'strip-double-wrap)))))))
 
-- 
2.36.0

[bug#55419] [PATCH 5/5] gnu: emacs-wide-int: Use new package style.

[Liliana Marie Prikler]

* gnu/packages/emacs.scm (emacs-wide-int)[arguments]: Convert to list of
G-Expressions.
[inputs]: Use modify-inputs.
---
 gnu/packages/emacs.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 392b3297b0..25dab7ef59 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -448,7 +448,7 @@ (define-public emacs-wide-int
     (arguments
      (substitute-keyword-arguments (package-arguments emacs)
        ((#:configure-flags flags)
-        `(cons "--with-wide-int" ,flags))))))
+        #~(cons "--with-wide-int" #$flags))))))
 
 (define-public guile-emacs
   (let ((commit "41120e0f595b16387eebfbf731fff70481de1b4b")
-- 
2.36.0

[bug#55419] [PATCH 0/5] Use new package style for Emacs variants.

[Liliana Marie Prikler]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1005 bytes --]

Hi Guix,

earlier "today" (or yesterday depending on your timezones), I pushed a
bunch of patches that update Emacs to 28.1 and also uses the new package
style.  Sadly this was not reflected in most Emacs variants leading to a
lot of failing builds.  The most important one – emacs-minimal – was
already fixed silently, this series ought to take care of the rest.

I'm currently building these locally, but chances are that for some of
you these changes will arrive too late by the time I'll be able to push
them, so I'm sharing them here for those with a faster machine or access
to the CI :)

Cheers

Liliana Marie Prikler (5):
  gnu: emacs-next-pgtk: Use new package style.
  gnu: emacs-xwidgets: Use new package style.
  gnu: emacs-no-x: Use new package style.
  gnu: emacs-no-x-toolkit: Use new package style.
  gnu: emacs-wide-int: Use new package style.

 gnu/packages/emacs.scm | 60 ++++++++++++++++++++----------------------
 1 file changed, 29 insertions(+), 31 deletions(-)

-- 
2.36.0

[bug#55419] [PATCH 0/5] Use new package style for Emacs variants.

[Maxime Devos]

[-- Attachment #1: Type: text/plain, Size: 88 bytes --]

The patches theirselves LGTM, but I did not build the packages.

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

bug#55419: [PATCH 0/5] Use new package style for Emacs variants.

[Liliana Marie Prikler]

Am Sonntag, dem 15.05.2022 um 10:43 +0200 schrieb Maxime Devos:
> The patches theirselves LGTM, but I did not build the packages.
Thanks.  The packages all built locally, so I pushed them.

Cheers