feeley / define-library Goto Github PK

Library:

(define-library (spheres/math/exact)
    (export double) ; <-- If commented, 
    (import (gambit))
    (begin
      (define double (λ (x) (* 2 x)))))

Code using the library:

(define-library (asdf) (import (gambit) (spheres/math/exact)) (begin (pp 'asdf)))

If (export) isn't used, then:

*** ERROR IN (console)@10.71 -- Unbound variable: spheres/math/exact#pp

Should the (export) clause be mandatory? If so, a warning/error should be emitted when absent.

This bug can be reproduce with a library file (.sld) that has exactly zero characters in it, when importing it.

This might be intentional, but I believe that for seamless REPL integration of the new define-syntax form, this shouldn't happen:

> (load "define-library")
"/Users/Alvaro/define-library/define-library.o1"
> (define-library (asdf) (import (gambit)))
> (define-library (asdf) (import (gambit)))
*** ERROR IN (console)@3.2 -- Unbound variable: asdf#define-library
1> ,d

Setting namespace to empty solves it, if it is actually a bug.

> (##namespace (""))
> (define-library (asdf) (import (gambit)))

Just adding this form at the end of a define-syntax expansion should be fine, and seems to work in other tests.

Looks current implementation completely ignored syntactic closure / syntax-rules, will any of these be in the road map? what about define-macro?