Summary
sassy - broken tests and missing export
Metadata
- Id: 4a9ec1bba702ca2cb3f858787c2eec5b8dfe9703
- Trac id: 536
- Type: defect
- Reporter: sjamaan
- Owner:
- Cc:
- Status: closed
- Component: extensions
- Estimated difficulty:
- Resolution: fixed
- Priority: major
- Milestone:
- Version: 4.6.x
- Changetime: 2011-03-20 13:58:10 UTC
- Created: 2011-03-15 21:22:43 UTC
- Keywords: sassy
Attachments
- 4a9ec1bba702ca2cb3f858787c2eec5b8dfe9703/attachments/sassy.patch
Description
The Sassy egg fails to test properly. See http://tests.call-cc.org/experimental/openbsd/x86/2011/03/12/salmonella-report/tests/sassy.html
The fix is simple: the run.scm wrapper should not just call require-library but also import the libraries.
After applying this fix, I get another error:
Error: unbound variable: sassy-data-size
Call history:
<eval> [test-misc] (make-list 24 144)
<eval> [test-misc] (list 80)
<eval> [test-misc] (succeed "text-align")
<eval> [succeed] (display "test passed: ")
<eval> [succeed] (display x)
<eval> [succeed] (newline)
<eval> [direc-test] (test-data)
<eval> [test-data] (sassy (quote ((data (label foo (dwords "ab")) (align 8) (label bar (dwords 100 quux)) (dwords -3242......
<eval> [test-data] (= 40 (sassy-data-size o))
<eval> [test-data] (sassy-data-size o) <--
Looking at sassy-48.scm and comparing it with sassy-chicken.scm, it seems that sassy-data-size is simply missing from the export list.
Here's a patch that fixes both issues.