.asd file IN-PACKAGE

Robert P. Goldman rpgoldman at sift.info
Mon Jul 11 13:36:04 UTC 2016


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




More information about the iterate-devel mailing list