emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#59417: closed ([PATCH] gnu: Add emacs-on.)


From: GNU bug Tracking System
Subject: bug#59417: closed ([PATCH] gnu: Add emacs-on.)
Date: Mon, 21 Nov 2022 14:46:02 +0000

Your message dated Mon, 21 Nov 2022 14:40:58 +0000
with message-id <87o7t0tlh0.fsf@cbaines.net>
and subject line Re: [bug#59417] [PATCH] gnu: Add emacs-on.
has caused the debbugs.gnu.org bug report #59417,
regarding [PATCH] gnu: Add emacs-on.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
59417: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=59417
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] gnu: Add emacs-on. Date: Sun, 20 Nov 2022 12:39:31 -0600
* gnu/packages/emacs-xyz.scm (emacs-on): New variable.
---
 gnu/packages/emacs-xyz.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 5c1138e954..232a547fb0 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -33845,6 +33845,28 @@ (define-public emacs-mpv
 process via its IPC interface.")
       (license license:gpl3+))))
 
+(define-public emacs-on
+  (let ((commit "83baed07a105c135190d49d7ec77f95e8f38fa54")
+        (revision "0"))
+    (package
+      (name "emacs-on")
+      (version (git-version "0.1.0" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/ajgrf/on.el";)
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "14ly3w89hf4jgd8wacirxgd8vrgj7vc0hpfn911w5wa4k2a6nzpp"))))
+      (build-system emacs-build-system)
+      (home-page "https://github.com/ajgrf/on.el";)
+      (synopsis "Hooks for faster Emacs startup")
+      (description
+"This package exposes a number of utility hooks and functions ported from Doom 
Emacs.")
+      (license license:expat))))
+
 (define-public emacs-waveform
   ;; XXX: Upstream provides no Version keyword.  Using 0 as base version.
   (let ((commit "ee52c6a72b3e9890743e3a6e2fc1f3195f5687b2")
-- 
2.38.1




--- End Message ---
--- Begin Message --- Subject: Re: [bug#59417] [PATCH] gnu: Add emacs-on. Date: Mon, 21 Nov 2022 14:40:58 +0000 User-agent: mu4e 1.8.11; emacs 28.2
jgart via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-on): New variable.
> ---
>  gnu/packages/emacs-xyz.scm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)

Thanks, I've pushed this to master as
f0cc7542f535cd0ccb75352cabc465a8dc1b1551.

Note that I did move the package definition to not be so close to the
bottom of the module. Thanks for following the comment and not putting
it right at the bottom, but the other important bit is "place them above
by existing packages with similar functionality or similar names".

The intent with that is to avoid Git conflicts when several packages are
added in the same place. For example, you also sent this patch
https://issues.guix.gnu.org/59429 that adds emacs-ukrainian-holidays in
the same place. That's just asking for Git trouble!

I normally just search for the start of the package name, e.g. emacs-o
and then find a similarly named package to place the new one by,
e.g. emacs-on-screen in this case.

Chris

Attachment: signature.asc
Description: PGP signature


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]