Hi Sebastian!

I'm sorry to come to you so late, real work and life took precedence...

On Mon, 28 Apr 2008 01:02:06 +0200, Sebastian Bober wrote:
> the bug is caused by the specifications in debian/cl-usocket.install
> that overwrites the original usocket.asd with a symlink from the
> "test" directory.

It seems the problem is deeper than that.  In fact, after having
corrected the usocket.asd bug you get another error:

=====
[EMAIL PROTECTED]:~$ sbcl
* (require 'usocket)
[...]

; compiling file "/usr/share/common-lisp/source/cl-usocket/package.lisp" 
(written 19 DEC 2007 07:13:00 AM):
; compiling (IN-PACKAGE :CL-USER)
; compiling (DEFPACKAGE :USOCKET-TEST ...)
debugger invoked on a SB-KERNEL:SIMPLE-PACKAGE-ERROR in thread #<THREAD 
"initial thread" {1002704A71}>:
  The name "REGRESSION-TEST" does not designate any package.
=====

This error is caused by...

> In the attached patch the .install file is cut down to the necessary
> files. The test suite itself is not that interesting and including it
> causes file name overlaps ("package.lisp" is in usocket and
> usocket-test).

...the installed "package.lisp" coming from usocket/test/ instead of
usocket/, but correcting also this causes:

=====
[EMAIL PROTECTED]:~$ sbcl
* (require 'usocket)
[...]

; /var/cache/common-lisp-controller/1000/sbcl/cl-usocket/usocket.fasl written
; compilation finished in 0:00:00

debugger invoked on a SB-INT:SIMPLE-FILE-ERROR in thread #<THREAD "initial 
thread" {1002704A71}>:
  failed to find the TRUENAME of 
/usr/share/common-lisp/source/cl-usocket/condition.lisp:
    No such file or directory
=====

So there are more bugs than the single ASDF system definition missing.

The first problem relies in debian/cl-usocket.install: since we want to
install the whole folders, we should omit the '*'.  This solves the
usocket.asd and package.lisp problems, not the missing condition.lisp
one, because this is not listed in debian/cl-usocket.install.

The following git commit completely fixes this bug:

http://git.debian.org/?p=pkg-common-lisp/cl-usocket.git;a=commit;h=d4f1ee5ea09a550774df7ca218d48d4541b289ce

> Another way would be to have usocket-test its own directory under
> /usr/share/common-lisp/source, but I really don't think thats
> necessary.

I'm always in favor of providing a test suite, if available, and
especially if it's minimal (in this case usocket/test/ is only 44K).
This is why I didn't remove it :-)

Thx, bye,
Gismo / Luca

Attachment: pgpz1UWHkG1ao.pgp
Description: PGP signature

Reply via email to