On 7/11/16 Jul 11 -5:20 AM, Attila Lendvai wrote:
>> 2. simply remove the IN-PACKAGE form altogether, and ASDF will
>> automagically put it into an appropriate package.
>
> i've done this with some other cleanups and pushed the changes into
> the darcs repo.
>
> let me know if there's anything else to be done.
>
FWIW, I think the definition for the test op can be simplified.
It's enough to put
:in-order-to (test-op (test-op "iterate/tests"))
into the definition of iterate (the test-op has a built-in dependence on
load-op for the same system), and then change the perform to
(defmethod asdf:perform ((op asdf:test-op) (c (eql (find-system
':iterate/tests))))
(funcall (intern "DO-TESTS" (find-package #+sbcl "SB-RT"
#-sbcl "REGRESSION-TEST"))))
That isn't actually a necessary change, but it's a lot tidier. The
current framework has the oddity that
(asdf:test-op "iterate/tests")
won't do anything.
It might even be possible to do away with the DEFMETHOD and put a
:METHOD form in the defsystem for iterate/tests, but even I'm a little
unsure about the syntax and semantics of that....
Best,
r
