GNU bug report logs - #59417
[PATCH] gnu: Add emacs-on.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 59417 in the body.
You can then email your comments to 59417 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: guix-patches <at> gnu.org
Cc: jgart <jgart <at> dismail.de>
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

Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: jgart <jgart <at> dismail.de>
Cc: 59417-done <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#59417] [PATCH] gnu: Add emacs-on.
Date: Mon, 21 Nov 2022 14:40:58 +0000

[Message part 1 (text/plain, inline)]

jgart via Guix-patches via <guix-patches <at> 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

[signature.asc (application/pgp-signature, inline)]

Message #16 received at submit <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 59417-done <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#59417] [PATCH] gnu: Add emacs-on.
Date: Sun, 27 Nov 2022 12:23:16 -0600

On Mon, 21 Nov 2022 14:40:58 +0000 Christopher Baines <mail <at> cbaines.net> wrote:
> Thanks, I've pushed this to master as
> f0cc7542f535cd0ccb75352cabc465a8dc1b1551.

Thanks!

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

Sorry about that! I'll keep it in mind for future patch sets. I think I'll
just start putting stuff in alphabetical order to avoid this altogether.

all best,

jgart

Message #22 received at 59417 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "jgart" <jgart <at> dismail.de>, "Christopher Baines" <mail <at> cbaines.net>
Cc: 59417 <at> debbugs.gnu.org, 59417-done <at> debbugs.gnu.org
Subject: Re: [bug#59417] [PATCH] gnu: Add emacs-on.
Date: Sun, 27 Nov 2022 18:26:10 +0000

[Message part 1 (text/plain, inline)]

On Sun Nov 27, 2022 at 6:23 PM GMT, jgart via Guix-patches via wrote:
> Sorry about that! I'll keep it in mind for future patch sets. I think I'll
> just start putting stuff in alphabetical order to avoid this altogether.

It doesn't mean alphabetical order by "similar names", so I'm pretty sure that
would still be an incorrect ordering.

    -- (

[signature.asc (application/pgp-signature, inline)]

Message #28 received at 59417 <at> debbugs.gnu.org (full text, mbox):

From: jgart <jgart <at> dismail.de>
To: "(" <paren <at> disroot.org>
Cc: 59417-done <at> debbugs.gnu.org, Christopher Baines <mail <at> cbaines.net>,
 59417 <at> debbugs.gnu.org
Subject: Re: [bug#59417] [PATCH] gnu: Add emacs-on.
Date: Sun, 27 Nov 2022 12:32:28 -0600

On Sun, 27 Nov 2022 18:26:10 +0000 "(" <paren <at> disroot.org> wrote:
> On Sun Nov 27, 2022 at 6:23 PM GMT, jgart via Guix-patches via wrote:
> > Sorry about that! I'll keep it in mind for future patch sets. I think I'll
> > just start putting stuff in alphabetical order to avoid this altogether.
> 
> It doesn't mean alphabetical order by "similar names", so I'm pretty sure that
> would still be an incorrect ordering.

I was thinking strict alphabetical ordering of packages like crates-io.scm

But I realize now that a given module that is not crates-io.scm might
have packages in random order...

Message #34 received at 59417 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "jgart" <jgart <at> dismail.de>
Cc: 59417-done <at> debbugs.gnu.org, Christopher Baines <mail <at> cbaines.net>,
 59417 <at> debbugs.gnu.org
Subject: Re: [bug#59417] [PATCH] gnu: Add emacs-on.
Date: Sun, 27 Nov 2022 18:51:51 +0000

[Message part 1 (text/plain, inline)]

On Sun Nov 27, 2022 at 6:32 PM GMT, jgart wrote:
> On Sun, 27 Nov 2022 18:26:10 +0000 "(" <paren <at> disroot.org> wrote:
> I was thinking strict alphabetical ordering of packages like crates-io.scm
>
> But I realize now that a given module that is not crates-io.scm might
> have packages in random order...

Yup, it's only the crates-*.scm files that should be alphabetically ordered, I think.

    -- (

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.