search

fukamachi / lack

Lack, the core of Clack
MIT License
147 stars 33 forks source link

Circular dependencies with Clack #54

Open Hellseher opened 3 years ago

Hellseher commented 3 years ago

Hi,

I tried to enabled tests during packing for Guix, but t-lack depends on Clack and Clack depends on Lack, which impossible to resolve during build and test stages.

https://github.com/fukamachi/clack/blob/master/clack.asd#L24

  :depends-on (:lack
               :lack-middleware-backtrace
               :lack-util
               :bordeaux-threads
               :usocket
               :alexandria
               :uiop)

https://github.com/fukamachi/lack/blob/master/t-lack.asd#L14

  :depends-on (:lack
               :clack
               :clack-v1-compat
               :prove)
fukamachi commented 3 years ago

Can it be resolved by just deleting :clack and :clack-v1-compat?

Hellseher commented 3 years ago

Hi,

I did not test it yet, it probably could resolve it when there is one way dependencies like:

clack

 :depends-on (:lack
              :lack-middleware-backtrace
              :lack-util
              :bordeaux-threads
              :usocket
              :alexandria
              :uiop)

lack's tests would depend just on 

  :depends-on (:lack
               :prove)