Opened 3 years ago

Closed 18 months ago

#1777 closed defect (fixed)

Drop builtin-features from eval.scm

Reported by: sjamaan Owned by:
Priority: major Milestone: 5.4
Component: core libraries Version: 5.2.0
Keywords: Cc:
Estimated difficulty: easy

Description

This seems like it's a hack from ye olden days when we didn't have a module system yet, and for example (require srfi-30) would otherwise try to attempt loading a unit library named srfi-30.

This is confusing and a bit weird, and also there's ##sys#features which might run out of sync with this features list.

To make this change, the srfi module registrations should be changed to be as part of the library unit. For example by changing register-primitive-module to use #f or 'library as the module library name.

NOTE: Check what the situation with chicken.csi and chicken.foreign is - can they also use #f or 'library?

Change History (2)

comment:1 Changed 3 years ago by sjamaan

I checked, and actually (require srfi-30) did give an error in 4.13. Also, SRFI-55 doesn't expand to require but to import in CHICKEN 5, so I think it won't get broken by this change, either.

comment:2 Changed 18 months ago by sjamaan

Resolution: fixed
Status: newclosed

Was fixed as part of 137ae4adb

Note: See TracTickets for help on using tickets.