Hi Jim,

The attached patch adds support for CHICKEN 6 to the rfc3339 egg.  It
breaks support for CHICKEN versions < 5.2.0, though.  In case you see
that as an issue, please let me know and I'll create a more portable
version of the patch.

I also volunteer to tag a new release, if that's more convenient to you.
Here I don't know how you want to proceed, though.  There are basically
two options:

1. Keep the CHICKEN 5 code unmodified and make new modifications to the
   code for CHICKEN 6 only (this would be like a fork of the egg).  The
   version of the egg for different versions of CHICKEN will most likely
   be different.

2. Use identical egg code for both CHICKEN 5 and CHICKEN 6.  In this
   case it would make sense to have the same egg versions for both
   versions of CHICKEN.

Your call.

rfc3339 is a dependency of salmonella-feeds, and I'm in the process of
porting the salmonella tools to CHICKEN 6.

All the best.
Mario
-- 
http://parenteses.org/mario
>From c7cfd4c8249fbd86d8c907fb38e4f92599b9ec26 Mon Sep 17 00:00:00 2001
From: Mario Domenech Goulart <[email protected]>
Date: Tue, 17 Dec 2024 20:13:12 +0100
Subject: [PATCH] Use set-record-printer! instead of define-record-printer

define-record-printer has been deprecated since CHICKEN 5.2.0 and is
not available in CHICKEN 6.

Use set-record-printer! instead (this breaks compatibility with
CHICKEN versions < 5.2.0).
---
 rfc3339.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/rfc3339.scm b/rfc3339.scm
index 768c035..cfea9a4 100644
--- a/rfc3339.scm
+++ b/rfc3339.scm
@@ -156,8 +156,9 @@
                          (c2 hh) ":" (c2 mm) ":" (c2 ss)
                          (cf fs) (tzstr tzoff)))))
 
-(define-record-printer (rfc3339 x out)
-  (fprintf out "#<rfc3339 ~s>" (rfc3339->string x)))
+(set-record-printer! rfc3339
+  (lambda (x out)
+    (fprintf out "#<rfc3339 ~s>" (rfc3339->string x))))
 
 ;;; convenience functions
 
-- 
2.39.5

Reply via email to