Summary

R7RS cond-expand library form is broken

Metadata

• Id: 8fd44181d66967bcae27559a92d9a897f3929fda
• Trac id: 1697
• Type: defect
• Reporter: zaifir
• Owner:
• Cc:
• Status: closed
• Component: unknown
• Estimated difficulty:
• Resolution: fixed
• Priority: major
• Milestone: someday
• Version: 5.2.0
• Changetime: 2020-06-16 15:01:52 UTC
• Created: 2020-04-18 19:27:08 UTC
• Keywords:

Attachments

• 8fd44181d66967bcae27559a92d9a897f3929fda/attachments/0001-Omit-C4-only-module-check-add-cond-expand-tests.patch
• 8fd44181d66967bcae27559a92d9a897f3929fda/attachments/0001-Omit-C4-specific-module-check.patch

Description

The `r7rs` egg's support for the `(library ...)` form in `cond-expand` is broken. Running the following example

 (import (r7rs))
 (define spiffy-installed?
   (cond-expand
     ((library (spiffy)) #t)
     (else #f)))

gives this error:

 Error: during expansion of (cond-expand ...) - unbound variable: ##sys#core-library-modules

Changes and comments

[2020-06-15 22:45:16 UTC] wasamasa attached 0001-Omit-C4-specific-module-check.patch (description=#f)

[2020-06-15 22:45:24 UTC] wasamasa wrote:

This particular error originates from two checks that are no longer necessary in C5 due to unification of modules and units. Here's a patch doing the necessary cond-expand.

[2020-06-15 23:00:25 UTC] wasamasa attached 0001-Omit-C4-only-module-check-add-cond-expand-tests.patch (description=#f)

[2020-06-16 15:01:52 UTC] felix changed status from new to closed

[2020-06-16 15:01:52 UTC] felix set resolution to fixed

[2020-06-16 15:01:52 UTC] felix wrote:

Patch applied and a new version has been tagged (1.0.2)