Opened 4 years ago
Closed 3 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 4 years ago by
comment:2 Changed 3 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 torequirebut toimportin CHICKEN 5, so I think it won't get broken by this change, either.