Opened 3 years ago
Closed 2 years 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
comment:2 Changed 2 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Was fixed as part of 137ae4adb
I checked, and actually
(require srfi-30)
did give an error in 4.13. Also, SRFI-55 doesn't expand torequire
but toimport
in CHICKEN 5, so I think it won't get broken by this change, either.