Bug#494404: patch + comments on cl-geodesics

Peter De Wachter pdewacht at gmail.com
Mon Aug 11 22:36:19 UTC 2008


cl-geodesics compiles for me with the attached patch.

One of the problems is that the ASDF system definition has unbalanced
parentheses. This makes it impossible to load/use this package. The
version in sarge and etch has the same bug, so it doesn't look like
anybody has used cl-geodesics in the past several years. It should
probably be removed rather than fixed.

OTOH, if you decide to keep it, there are a few other problems I
noticed:

- utilities.lisp doesn't have an IN-PACKAGE form, so it gets loaded in
whatever random package happens to be active.

- The GD-STATIC-EQUAL, GD-STATIC-UNEQUAL and GD-COSMOLOGICAL
systems each define the same symbols in the GEODESICS package, so they
can't be loaded simultaneously. But that's exactly what the GEODESICS
system does. I think each of the GD-* systems needs to live in a
separate package?

-- 
Peter De Wachter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cl-geodesics-asdf.patch
Type: text/x-patch
Size: 2114 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-common-lisp-devel/attachments/20080812/efd88b3d/attachment.bin 


More information about the pkg-common-lisp-devel mailing list