Opened 21 months ago
Closed 10 months ago
#1754 closed defect (fixed)
sandbox egg test fails on missing ##sys#eval-decorator procedure
Reported by: | Vasilij Schneidermann | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | someday |
Component: | extensions | Version: | 5.2.0 |
Keywords: | sandbox eval | Cc: | felix winkelmann, winkelmann |
Estimated difficulty: | easy |
Description
When running chicken-install -test sandbox
, the second to last test exercising the fuel limit fails with Error: unbound variable: ##sys#eval-decorator
. This can be resolved by replacing the respective call in the sandbox sources with the definition of eval-decorator
in core's eval.scm. It's unclear whether that will break compatibility with C4 though.
Attachments (2)
Change History (5)
comment:1 Changed 16 months ago by
comment:2 Changed 16 months ago by
Anyway I'm having trouble getting safe-environment-macro-set! to do what I want; not sure if this fix has something to do with it, or it's just my little experience writing any kind of macros. The docs are not at all clear on what the "procedure of one argument" ought to return. So I keep getting the "illegal non-atomic object" error for almost everything I try.
Changed 10 months ago by
Attachment: | 0001-Use-eval-decorator-from-C5.patch added |
---|
Changed 10 months ago by
Attachment: | 0001-Improve-test-suite-to-check-for-error-messages.patch added |
---|
comment:3 Changed 10 months ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
The patches have been applied (r41254)
Confirmed. The egg is not usable this way, AFAICT; I ran into it like this:
and your suggestion fixes it: