skribilo-bugs
[Top][All Lists]
Advanced

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

bug#54705: [PATCH] reader: Add Gemtext reader.


From: Ludovic Courtès
Subject: bug#54705: [PATCH] reader: Add Gemtext reader.
Date: Sun, 10 Apr 2022 22:11:29 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)

Hey!

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * src/guile/skribilo/reader/gemtext.scm: New file.
> * src/guile/Makefile.am (readers): Register it.
> * doc/user/syntax.skb (The Gemtext Syntax): New section.
> * tests/readers/gemtext.test: New file.
> * tests/Makefile.am (TESTS): Add readers/gemtext.test.

This looks great to me!

> +(define-module (skribilo reader gemtext)
> +  #:use-module (rnrs io ports)
> +  #:use-module (srfi srfi-26)
> +  #:use-module (srfi srfi-71)
> +  #:use-module (srfi srfi-171)

I’ve just realized that, currently, Skribilo supports Guile 2.x in
addition to 3.0, but neither 2.2 nor 2.0 has (srfi srfi-171).

Maybe the simplest option is to check for (srfi srfi-171) in
configure.ac and disable compilation of this module if it’s unavailable,
similar to the ‘BUILD_RSS2_READER’ conditional.

WDYT?

Minor cosmetic comments:

> +(skribilo-module-syntax)

I think you can omit this line and use regular #:keywords.

That’s all I have to say, thank you!

Ludo’.





reply via email to

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