Opened 10 months ago

Last modified 10 months ago

#1777 new defect

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


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 (1)

comment:1 Changed 10 months 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.

Note: See TracTickets for help on using tickets.