GNU bug report logs - #50374
[PATCH 0/1] Add org-roam-extensions package

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 50374 in the body.
You can then email your comments to 50374 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: "Collin J. Doering" <collin <at> rekahsoft.ca>
To: guix-patches <at> gnu.org
Subject: [PATCH 0/1] Add org-roam-extensions package
Date: Sat, 04 Sep 2021 12:12:30 -0400

Instead of the implementation put forth in
https://issues.guix.gnu.org/50333 this change includes org-roam
extensions as a separate package.

Collin J. Doering (1):
  gnu: Add emacs-org-roam-extensions

 gnu/packages/emacs-xyz.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

-- 
2.33.0


-- 
Collin J. Doering

http://rekahsoft.ca
http://blog.rekahsoft.ca
http://git.rekahsoft.ca

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

From: "Collin J. Doering" <collin <at> rekahsoft.ca>
To: 50374 <at> debbugs.gnu.org
Subject: [PATCH 1/1] gnu: Add emacs-org-roam-extensions
Date: Sat, 04 Sep 2021 12:19:38 -0400

---
 gnu/packages/emacs-xyz.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index f04a0225b3..830c642492 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -28264,6 +28264,25 @@ as a plug-and-play solution for anyone already using Org mode for their
 personal wiki.")
        (license license:gpl3+)))
 
+(define-public emacs-org-roam-extensions
+  (package
+    (inherit emacs-org-roam)
+    (name "emacs-org-roam-extensions")
+    (propagated-inputs
+     `(,@(package-propagated-inputs emacs-org-roam)
+       ("emacs-org-roam" ,emacs-org-roam)))
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'enter-subdirectory
+           (lambda _ (chdir "extensions") #t))
+         (add-before 'install-license-files 'leave-subdirectory
+           (lambda _ (chdir "..") #t)))))
+    (synopsis "Extensions for Org-Roam")
+    (description "Emacs Org Roam Extensions include utilities for daily note
+taking, the org-roam protocol, graphing note relationships and providing id link
+overlays.")))
+
 (define-public emacs-org-roam-bibtex
   (package
     (name "emacs-org-roam-bibtex")
-- 
2.33.0


-- 
Collin J. Doering

http://rekahsoft.ca
http://blog.rekahsoft.ca
http://git.rekahsoft.ca

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

From: Adolfo De Unánue <adolfo <at> unanue.mx>
To: 50374 <at> debbugs.gnu.org
Date: Sun, 12 Sep 2021 11:19:13 -0500

Hi

I am the author of the patch https://issues.guix.gnu.org/50333 (my 
first
patch, so I am a newbie on this).

Could you help me to understand why your proposal is a better 
solution?
I want to learn how to approach to this matters in the future.

My rationale is that the extensions folder *is* part of org-roam, 
that's
why I *added* them to the original package.

Also, Do we need to do something in order to get this patch (or 
mine)
approved?
(again, just asking because this workflow is new for me)

Thanks in advance for your time

--
- A

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

From: zimoun <zimon.toutoune <at> gmail.com>
To: "Collin J. Doering" <collin <at> rekahsoft.ca>
Cc: 50374 <at> debbugs.gnu.org
Subject: Re: [bug#50374] [PATCH 0/1] Add org-roam-extensions package
Date: Tue, 14 Sep 2021 14:30:08 +0200

Hi,

On Sat, 04 Sep 2021 at 12:12, "Collin J. Doering" <collin <at> rekahsoft.ca> wrote:
> Instead of the implementation put forth in
> https://issues.guix.gnu.org/50333 this change includes org-roam
> extensions as a separate package.

Naive question, why should the extensions be distributed as a separate
package?  Well, I am not an Org-Roam user… having the extensions along
with the package emacs-org-roam seem the best; other said, why an
Org-Roam user would be annoyed to have these extensions and not use
them?

All the best,
simon

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

From: "Collin J. Doering" <collin <at> rekahsoft.ca>
To: Adolfo De Unánue <adolfo <at> unanue.mx>
Cc: 50374 <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#50374] (no subject)
Date: Thu, 16 Sep 2021 09:38:31 -0400

Hi Adolfo,

I am not a guix expert either and am relatively new to the project. I
actually asked about this topic on guix IRC, where folks there suggested
splitting it into another package. I see very little difference between
including the org-roam-extensions as a separate package output, or as a
standalone package. I feel that having it as an additional output is
more "pure", but found that many packages already use a standalone
package for extensions, and I found that doing it in this way made both
packages themselves easier to define. You can see this for yourself with
a query like this `guix package -s '.*-extensions$' | recsel -CP name`.

Hope this helps, and kind regards!

On 12 Sep 2021 at 11:19, Adolfo De Unánue <adolfo <at> unanue.mx> wrote:

> Hi
>
> I am the author of the patch https://issues.guix.gnu.org/50333 (my first
> patch, so I am a newbie on this).
>
> Could you help me to understand why your proposal is a better solution?
> I want to learn how to approach to this matters in the future.
>
> My rationale is that the extensions folder *is* part of org-roam, that's
> why I *added* them to the original package.
>
> Also, Do we need to do something in order to get this patch (or mine)
> approved?
> (again, just asking because this workflow is new for me)
>
> Thanks in advance for your time


-- 
Collin J. Doering

http://rekahsoft.ca
http://blog.rekahsoft.ca
http://git.rekahsoft.ca

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

From: "Collin J. Doering" <collin <at> rekahsoft.ca>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 50374 <at> debbugs.gnu.org
Subject: Re: [bug#50374] [PATCH 0/1] Add org-roam-extensions package
Date: Thu, 16 Sep 2021 09:48:25 -0400

Hi Zimoun,

Thanks for taking the time to look at this patch.

On 14 Sep 2021 at 14:30, zimoun <zimon.toutoune <at> gmail.com> wrote:

> Hi,
>
> On Sat, 04 Sep 2021 at 12:12, "Collin J. Doering" <collin <at> rekahsoft.ca> wrote:
>> Instead of the implementation put forth in
>> https://issues.guix.gnu.org/50333 this change includes org-roam
>> extensions as a separate package.
>
> Naive question, why should the extensions be distributed as a separate
> package?

They are not required for the use of org-roam. They could also be
distributed as an additional output on the org-roam package itself.
Happy to re-adjust if that's preferred.

> Well, I am not an Org-Roam user… having the extensions along with the
> package emacs-org-roam seem the best; other said, why an Org-Roam user
> would be annoyed to have these extensions and not use them?

I don't think they need to be included directly in the main output of
the emacs-org-roam package as not all org-roam users will leverage them.
However I can't think of any reasons why a org-roam user would be
annoyed that they are included (other then including software that is
not used, which is not necessarily favorable).

Please let me know your preference between the following:

1. org-roam extensions as part of a separate package (this is what is
already presented in this patch set). 
2. Extensions as a separate `extensions` output on the emacs-org-roam package
3. Build the extensions directly into the primary output of the
emacs-org-roam package (this is already done in https://issues.guix.gnu.org/50333)

> All the best,

You too!

-- 
Collin J. Doering

http://rekahsoft.ca
http://blog.rekahsoft.ca
http://git.rekahsoft.ca

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

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: "Collin J. Doering" <collin <at> rekahsoft.ca>
Cc: 50374-done <at> debbugs.gnu.org
Subject: Re: [bug#50374] [PATCH 0/1] Add org-roam-extensions package
Date: Mon, 17 Jan 2022 00:31:24 +0100

Hello,

"Collin J. Doering" <collin <at> rekahsoft.ca> writes:

> Instead of the implementation put forth in
> https://issues.guix.gnu.org/50333 this change includes org-roam
> extensions as a separate package.

Thank you for the patch.

However, extensions for Org Roam are installed since bug#50287.

Therefore, I'm closing this bug report. Feel free to re-open it if you
think something is missing.

Regards,
-- 
Nicolas Goaziou

Previous Next

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