diff mbox series

[bug#61147] gnu: emacs-org-board: Not dependent on emacs-org

Message ID 2f8fc98b2e4b25035625a72b75c63c166427ef51.1674993544.git.csantosb@inventati.org
State New
Headers show
Series [bug#61147] gnu: emacs-org-board: Not dependent on emacs-org | expand

Commit Message

Cayetano Santos Jan. 29, 2023, 11:59 a.m. UTC 
* gnu/packages/emacs-xyz.scm (emacs-org-board): remove emacs-org

Org is already included with emacs.
---
 gnu/packages/emacs-xyz.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


base-commit: b9e6e31877cdb96cceba4d1ec6268f86b824dec4

Comments

Nicolas Goaziou Jan. 29, 2023, 2:50 p.m. UTC | #1 
Hello,

Cayetano Santos via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-org-board): remove emacs-org
>
> Org is already included with emacs.

This is a fair point, and there are multiple occurrences of similar
situations in our code base (with, e.g., emacs-seq).

However, I think there is no consensus at the moment; it is not clear if
we should use the bundled package, or the latest one. I lean towards the
second idea, as long as our packages are up-to-date. In any case,
you may want to raise the subject on the devel mailing list instead of
trying to changes status package-wise.

WDYT?

Regards,
dziltener--- via Guix-patches via Jan. 29, 2023, 6:54 p.m. UTC | #2 
>dim. 29 janv. 2023 at 15:50, Nicolas Goaziou 
><mail@nicolasgoaziou.fr> ...

> Hello,
>
> Cayetano Santos via Guix-patches via <guix-patches@gnu.org> 
> writes:
>
>> * gnu/packages/emacs-xyz.scm (emacs-org-board): remove 
>> emacs-org
>>
>> Org is already included with emacs.
>
> This is a fair point, and there are multiple occurrences of 
> similar
> situations in our code base (with, e.g., emacs-seq).
>
> However, I think there is no consensus at the moment; it is not 
> clear if
> we should use the bundled package, or the latest one. I lean 
> towards the
> second idea, as long as our packages are up-to-date. In any 
> case,
> you may want to raise the subject on the devel mailing list 
> instead of
> trying to changes status package-wise.
>
> WDYT?

You’re right. Devel list is the place to discuss about this before 
going
any further.
Nicolas Goaziou Jan. 29, 2023, 7:47 p.m. UTC | #3 
Hello,

Cayetano Santos via Guix-patches via <guix-patches@gnu.org> writes:

> You’re right. Devel list is the place to discuss about this before
> going
> any further.

OK. Then, for the time being, I'm closing this bug report. You can
always re-open it depending on how it turns out on the devel ML.
diff mbox series

Patch

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 4b2fc8f0dc..92a11e18b5 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -25142,7 +25142,7 @@  (define-public emacs-org-board
       (inputs
        (list wget))
       (propagated-inputs
-       (list emacs-org emacs-ztree))
+       (list emacs-ztree))
       (home-page "https://github.com/charlesroelli/org-board")
       (synopsis "Bookmarking and web archival system for Org mode")
       (description