GNU bug report logs - #78086
[PATCH] gnu: Add emacs-org-rss-publishing.

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 78086 in the body.
You can then email your comments to 78086 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: Fabio Natali <me <at> fabionatali.com>
To: guix-patches <at> gnu.org
Cc: Fabio Natali <me <at> fabionatali.com>
Subject: [PATCH] gnu: Add emacs-org-rss-publishing.
Date: Sun, 27 Apr 2025 10:49:54 +0100

* gnu/packages/emacs-xyz.scm (emacs-org-rss-publishing): New variable.

Change-Id: I063fa0ba2ba3d87bcde2af8c8e3d04423de4ba2e
---
Hi All. 👋

This is to add 'org-rss-publishing', an RSS generator for Org Mode's built-in
publishing system. See the project page for details:

https://git.sr.ht/~taingram/org-publish-rss

I hope it looks alright, but let me know what you think and if there's anything
that needs to be addressed.

Thanks, cheers, Fabio.


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

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 6d10fb7737..0e0ffef58c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -129,7 +129,7 @@
 ;;; Copyright © 2023, 2025 Evgeny Pisemsky <mail <at> pisemsky.site>
 ;;; Copyright © 2023 Gabriel Wicki <gabriel <at> erlikon.ch>
 ;;; Copyright © 2022-2023 Simon Josefsson <simon <at> josefsson.org>
-;;; Copyright © 2023 Fabio Natali <me <at> fabionatali.com>
+;;; Copyright © 2023, 2025 Fabio Natali <me <at> fabionatali.com>
 ;;; Copyright © 2023 Arnaud Lechevallier <arnaud.lechevallier <at> free.fr>
 ;;; Copyright © 2023 Ahmad Draidi <a.r.draidi <at> redscript.org>
 ;;; Copyright © 2023 Sergiu Ivanov <sivanov <at> colimite.fr>
@@ -31854,6 +31854,32 @@ (define-public emacs-org-web-tools
 inserts it as an Org entry instead of displaying it in a new buffer.")
     (license license:gpl3+)))
 
+(define-public emacs-org-rss-publishing
+  (let ((commit "1b33dc252cfedfbb2107ea407266b168b71bebb1")
+        (revision "0"))
+    (package
+      (name "emacs-org-rss-publishing")
+      (version (git-version "0.0.0" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://git.sr.ht/~taingram/org-publish-rss")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "0axz3r8jw15imhfmbriljsiaxiiqhgrq77nad3kmmgpmvhhlk65f"))))
+      (build-system emacs-build-system)
+      (propagated-inputs (list emacs-org))
+      (home-page "https://git.sr.ht/~taingram/org-publish-rss")
+      (synopsis "RSS generator for Org Mode's built-in publishing system")
+      (description "This package adds new RSS generation options to the
+@code{org-publish-project-alist} variable (see the Org manual if you are new to
+the publishing options). It adds @code{:auto-rss} and other options that work
+similar to the included @code{:auto-sitemap} functionality. This should make it
+easy for users to add RSS feeds to existing Org-based websites.")
+      (license license:gpl3))))
+
 (define-public emacs-blimp
   (let ((commit "e420763d6d18b5d1be552cdbc47f91418343db03"))
     (package

base-commit: 7e5913f90df916d8d9f5c509354d62324f54f481
-- 
2.49.0

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

From: Cayetano Santos <csantosb <at> inventati.org>
To: 78086 <at> debbugs.gnu.org
Cc: ian <at> retrospec.tv, cox.katherine.e+guix <at> gmail.com, csantosb <at> inventati.org,
 andrew <at> trop.in, hako <at> ultrarare.space, divya <at> subvertising.org,
 me <at> fabionatali.com, liliana.prikler <at> gmail.com
Subject: Re: [bug#78086] [PATCH] gnu: Add emacs-org-rss-publishing.
Date: Sun, 27 Apr 2025 13:26:47 +0200

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

Hi Fabio,

  A few remarks:

> +      (version (git-version "0.0.0" revision commit))

Version is 0.6, see header in org-publish-rss.el.

> +      (propagated-inputs (list emacs-org))

No need to, emacs includes org.

> +      (license license:gpl3))))

gpl3+ ?

The, try a:

    ./pre-inst-env guix lint emacs-org-rss-publishing

you’ll get:

    sentences in description should be followed by two spaces; possible
    infractions at 155, 268

And don’t forget disable tests, if there are no, as this becomes mandatory:

+    (arguments `(#:tests? #false))      ;no tests

Could you please send a v2 ?

C.

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

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

From: Fabio Natali <me <at> fabionatali.com>
To: 78086 <at> debbugs.gnu.org
Cc: Fabio Natali <me <at> fabionatali.com>
Subject: [PATCH v2] gnu: Add emacs-org-rss-publishing.
Date: Sun, 27 Apr 2025 19:38:30 +0100

* gnu/packages/emacs-xyz.scm (emacs-org-rss-publishing): New variable.

Change-Id: I063fa0ba2ba3d87bcde2af8c8e3d04423de4ba2e
---
Hi,

Here's a v2 that should (hopefully) address all of Cayetano's points:

- Version now reported as 0.6.
- 'emacs-org' removed from the list of propagated inputs.
- Licence now reported as 'gpl3+'.
- Description now using double-spaces as sentence delimiters.
- Tests disabled.

Thanks for the useful feedback! I hope this looks better now.

Best wishes, Fabio.

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

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 6d10fb7737..0e0ffef58c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -129,7 +129,7 @@
 ;;; Copyright © 2023, 2025 Evgeny Pisemsky <mail <at> pisemsky.site>
 ;;; Copyright © 2023 Gabriel Wicki <gabriel <at> erlikon.ch>
 ;;; Copyright © 2022-2023 Simon Josefsson <simon <at> josefsson.org>
-;;; Copyright © 2023 Fabio Natali <me <at> fabionatali.com>
+;;; Copyright © 2023, 2025 Fabio Natali <me <at> fabionatali.com>
 ;;; Copyright © 2023 Arnaud Lechevallier <arnaud.lechevallier <at> free.fr>
 ;;; Copyright © 2023 Ahmad Draidi <a.r.draidi <at> redscript.org>
 ;;; Copyright © 2023 Sergiu Ivanov <sivanov <at> colimite.fr>
@@ -31854,6 +31854,32 @@ (define-public emacs-org-web-tools
 inserts it as an Org entry instead of displaying it in a new buffer.")
     (license license:gpl3+)))
 
+(define-public emacs-org-rss-publishing
+  (let ((commit "1b33dc252cfedfbb2107ea407266b168b71bebb1")
+        (revision "0"))
+    (package
+      (name "emacs-org-rss-publishing")
+      (version (git-version "0.0.0" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://git.sr.ht/~taingram/org-publish-rss")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "0axz3r8jw15imhfmbriljsiaxiiqhgrq77nad3kmmgpmvhhlk65f"))))
+      (build-system emacs-build-system)
+      (propagated-inputs (list emacs-org))
+      (home-page "https://git.sr.ht/~taingram/org-publish-rss")
+      (synopsis "RSS generator for Org Mode's built-in publishing system")
+      (description "This package adds new RSS generation options to the
+@code{org-publish-project-alist} variable (see the Org manual if you are new to
+the publishing options). It adds @code{:auto-rss} and other options that work
+similar to the included @code{:auto-sitemap} functionality. This should make it
+easy for users to add RSS feeds to existing Org-based websites.")
+      (license license:gpl3))))
+
 (define-public emacs-blimp
   (let ((commit "e420763d6d18b5d1be552cdbc47f91418343db03"))
     (package

base-commit: 7e5913f90df916d8d9f5c509354d62324f54f481
-- 
2.49.0

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

From: Fabio Natali <me <at> fabionatali.com>
To: 78086 <at> debbugs.gnu.org
Cc: Fabio Natali <me <at> fabionatali.com>
Subject: [PATCH v3] gnu: Add emacs-org-rss-publishing.
Date: Sun, 27 Apr 2025 19:51:59 +0100

* gnu/packages/emacs-xyz.scm (emacs-org-rss-publishing): New variable.

Change-Id: Ia1b02e050c42b1edcff6ad94a9599c87170e1a12
---
Ooops! I messed things up and sent a v2 which is identical to v1! Here's the new
patch with the various fixes. Hope it's alright. Thanks, cheers, Fabio.

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

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 6d10fb7737..a9478a5afa 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -129,7 +129,7 @@
 ;;; Copyright © 2023, 2025 Evgeny Pisemsky <mail <at> pisemsky.site>
 ;;; Copyright © 2023 Gabriel Wicki <gabriel <at> erlikon.ch>
 ;;; Copyright © 2022-2023 Simon Josefsson <simon <at> josefsson.org>
-;;; Copyright © 2023 Fabio Natali <me <at> fabionatali.com>
+;;; Copyright © 2023, 2025 Fabio Natali <me <at> fabionatali.com>
 ;;; Copyright © 2023 Arnaud Lechevallier <arnaud.lechevallier <at> free.fr>
 ;;; Copyright © 2023 Ahmad Draidi <a.r.draidi <at> redscript.org>
 ;;; Copyright © 2023 Sergiu Ivanov <sivanov <at> colimite.fr>
@@ -31854,6 +31854,31 @@ (define-public emacs-org-web-tools
 inserts it as an Org entry instead of displaying it in a new buffer.")
     (license license:gpl3+)))
 
+(define-public emacs-org-rss-publishing
+  (package
+    (name "emacs-org-rss-publishing")
+    (version "0.6")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://git.sr.ht/~taingram/org-publish-rss")
+             (commit "1b33dc252cfedfbb2107ea407266b168b71bebb1")))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "0axz3r8jw15imhfmbriljsiaxiiqhgrq77nad3kmmgpmvhhlk65f"))))
+    (build-system emacs-build-system)
+    (arguments `(#:tests? #false)) ;no tests
+    (home-page "https://git.sr.ht/~taingram/org-publish-rss")
+    (synopsis "RSS generator for Org Mode's built-in publishing system")
+    (description "This package adds new RSS generation options to the
+@code{org-publish-project-alist} variable (see the Org manual if you are new to
+the publishing options).  It adds @code{:auto-rss} and other options that work
+similar to the included @code{:auto-sitemap} functionality.  This should make it
+easy for users to add RSS feeds to existing Org-based websites.")
+    (license license:gpl3+)))
+
 (define-public emacs-blimp
   (let ((commit "e420763d6d18b5d1be552cdbc47f91418343db03"))
     (package

base-commit: 7e5913f90df916d8d9f5c509354d62324f54f481
-- 
2.49.0

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

From: Cayetano Santos <csantosb <at> inventati.org>
To: Fabio Natali <me <at> fabionatali.com>
Cc: Ian Eure <ian <at> retrospec.tv>,
 Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>,
 Andrew Tropin <andrew <at> trop.in>, Hilton Chain <hako <at> ultrarare.space>,
 Divya Ranjan Pattanaik <divya <at> subvertising.org>, 78086 <at> debbugs.gnu.org
Subject: Re: [bug#78086] [PATCH v3] gnu: Add emacs-org-rss-publishing.
Date: Mon, 28 Apr 2025 09:28:20 +0200

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

>dim. 27 avril 2025 at 19:51, Fabio Natali <me <at> fabionatali.com> wrote:

> * gnu/packages/emacs-xyz.scm (emacs-org-rss-publishing): New variable.
>
> Change-Id: Ia1b02e050c42b1edcff6ad94a9599c87170e1a12
> ---
> Ooops! I messed things up and sent a v2 which is identical to v1! Here's the new
> patch with the various fixes. Hope it's alright. Thanks, cheers, Fabio.

The let binding was fine, no need to remove it. Have a look at
emacs-0x0, for example, you can use it as a template (adding the
#:tests? flag)

C.

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

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

From: Fabio Natali <me <at> fabionatali.com>
To: 78086 <at> debbugs.gnu.org
Cc: Fabio Natali <me <at> fabionatali.com>
Subject: [PATCH v4] gnu: Add emacs-org-rss-publishing.
Date: Mon, 28 Apr 2025 10:13:45 +0100

* gnu/packages/emacs-xyz.scm (emacs-org-rss-publishing): New variable.

Change-Id: Ia1b02e050c42b1edcff6ad94a9599c87170e1a12
---
Ooops, thanks Cayetano. I followed emacs-0x0 as close as possible, I hope I got
it right. Thanks for all the help. 🙏🙏🙏

F.

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

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 6d10fb7737..dadb1fe9f7 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -129,7 +129,7 @@
 ;;; Copyright © 2023, 2025 Evgeny Pisemsky <mail <at> pisemsky.site>
 ;;; Copyright © 2023 Gabriel Wicki <gabriel <at> erlikon.ch>
 ;;; Copyright © 2022-2023 Simon Josefsson <simon <at> josefsson.org>
-;;; Copyright © 2023 Fabio Natali <me <at> fabionatali.com>
+;;; Copyright © 2023, 2025 Fabio Natali <me <at> fabionatali.com>
 ;;; Copyright © 2023 Arnaud Lechevallier <arnaud.lechevallier <at> free.fr>
 ;;; Copyright © 2023 Ahmad Draidi <a.r.draidi <at> redscript.org>
 ;;; Copyright © 2023 Sergiu Ivanov <sivanov <at> colimite.fr>
@@ -31854,6 +31854,32 @@ (define-public emacs-org-web-tools
 inserts it as an Org entry instead of displaying it in a new buffer.")
     (license license:gpl3+)))
 
+(define-public emacs-org-rss-publishing
+  (let ((commit "1b33dc252cfedfbb2107ea407266b168b71bebb1")
+        (revision "0"))
+    (package
+      (name "emacs-org-rss-publishing")
+      (version (git-version "0.6" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://git.sr.ht/~taingram/org-publish-rss")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "0axz3r8jw15imhfmbriljsiaxiiqhgrq77nad3kmmgpmvhhlk65f"))))
+      (build-system emacs-build-system)
+      (arguments `(#:tests? #false)) ;no tests
+      (home-page "https://git.sr.ht/~taingram/org-publish-rss")
+      (synopsis "RSS generator for Org Mode's built-in publishing system")
+      (description "This package adds new RSS generation options to the
+@code{org-publish-project-alist} variable (see the Org manual if you are new to
+the publishing options).  It adds @code{:auto-rss} and other options that work
+similar to the included @code{:auto-sitemap} functionality.  This should make it
+easy for users to add RSS feeds to existing Org-based websites.")
+      (license license:gpl3+))))
+
 (define-public emacs-blimp
   (let ((commit "e420763d6d18b5d1be552cdbc47f91418343db03"))
     (package

base-commit: 7e5913f90df916d8d9f5c509354d62324f54f481
-- 
2.49.0

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

From: Cayetano Santos <cayetano.santos <at> inventati.org>
To: Fabio Natali <me <at> fabionatali.com>
Cc: Ian Eure <ian <at> retrospec.tv>,
 Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>,
 Cayetano Santos <csantosb <at> inventati.org>, Andrew Tropin <andrew <at> trop.in>,
 Hilton Chain <hako <at> ultrarare.space>,
 Divya Ranjan Pattanaik <divya <at> subvertising.org>, 78086 <at> debbugs.gnu.org,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Subject: Re: [bug#78086] [PATCH v4] gnu: Add emacs-org-rss-publishing.
Date: Mon, 28 Apr 2025 22:41:40 +0200

>lun. 28 avril 2025 at 10:13, Fabio Natali <me <at> fabionatali.com> wrote:

> * gnu/packages/emacs-xyz.scm (emacs-org-rss-publishing): New variable.
>
> Change-Id: Ia1b02e050c42b1edcff6ad94a9599c87170e1a12
> ---
> Ooops, thanks Cayetano. I followed emacs-0x0 as close as possible, I hope I got
> it right. Thanks for all the help. 🙏🙏🙏

The package looks good to me !

Next time, watch your Change-Id, which is supposed to remain the same
with successive versions.

C.

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

From: Fabio Natali <me <at> fabionatali.com>
To: Cayetano Santos <cayetano.santos <at> inventati.org>
Cc: Ian Eure <ian <at> retrospec.tv>,
 Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>,
 Cayetano Santos <csantosb <at> inventati.org>, Andrew Tropin <andrew <at> trop.in>,
 Hilton Chain <hako <at> ultrarare.space>,
 Divya Ranjan Pattanaik <divya <at> subvertising.org>, 78086 <at> debbugs.gnu.org,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Subject: Re: [bug#78086] [PATCH v4] gnu: Add emacs-org-rss-publishing.
Date: Mon, 28 Apr 2025 21:54:10 +0100

On 2025-04-28, 22:41 +0200, Cayetano Santos <cayetano.santos <at> inventati.org> wrote:
> The package looks good to me !

Yay! Thanks Cayetano.

> Next time, watch your Change-Id, which is supposed to remain the same
> with successive versions.

Ha, got it. Thanks.

🙏

F.

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

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: 78086-done <at> debbugs.gnu.org
Cc: Cayetano Santos <cayetano.santos <at> inventati.org>,
 Fabio Natali <me <at> fabionatali.com>, Cayetano Santos <csantosb <at> inventati.org>
Subject: Re: [PATCH] gnu: Add emacs-org-rss-publishing.
Date: Fri, 02 May 2025 22:13:38 +0100

Thanks for the review, Cayetano! And, thanks for the patch, Fabio!

Pushed as 903916f8341bc45408f3883a448f7b0a04368796.

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

From: Fabio Natali <me <at> fabionatali.com>
To: Arun Isaac <arunisaac <at> systemreboot.net>
Cc: Cayetano Santos <csantosb <at> inventati.org>, 78086-done <at> debbugs.gnu.org
Subject: Re: [PATCH] gnu: Add emacs-org-rss-publishing.
Date: Fri, 02 May 2025 22:21:29 +0100

On 2025-05-02, 22:13 +0100, Arun Isaac <arunisaac <at> systemreboot.net> wrote:
> Thanks for the review, Cayetano! And, thanks for the patch, Fabio!

Super, thanks Arun! 🙏

F.

Previous Next

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