From fahree at gmail.com  Thu Feb 21 03:43:11 2013
From: fahree at gmail.com (=?ISO-8859-1?Q?Far=E9?=)
Date: Wed, 20 Feb 2013 22:43:11 -0500
Subject: [mcclim-devel] mcclim failure with asdf3
Message-ID: <CAN7nBXcJfKDqp0CxPh5oEbM6HRAJrahoJKcNJK52L1Hi95n6qg@mail.gmail.com>

ASDF3 now checks deferred warnings by default, which notably catches
forward references to special variables.

??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
If the Bible proves that God exists then comic books prove the existence
of Superman.    ? seen on #Atheism IRC

; file: /home/testgrid/quicklisp-patched2/dists/quicklisp/software/mcclim-20120909-cvs/graphics.lisp
; in: CLIM-EXTENSIONS:DEFINE-BITMAP-FILE-READER :XPM
;     (CLIM-INTERNALS::XPM-PARSE-FILE PATHNAME)
;
; caught STYLE-WARNING:
;   undefined function: CLIM-INTERNALS::XPM-PARSE-FILE
;
; compilation unit finished
;   Undefined types:
;     CLIM:CLIM-STREAM-PANE CLIM:STREAM-OUTPUT-HISTORY-MIXIN
CLIM-INTERNALS::TOP-LEVEL-SHEET-PANE
;   Undefined functions:
;     (SETF CLIM-BACKEND:PORT-MOTION-HINTS) (SETF
CLIM-BACKEND:SELECTION-OWNER) (SETF CLIM-BACKEND:SELECTION-TIMESTAMP)
CLIM-LISP:DESCRIBE CLIM:MAKE-SPACE-REQUIREMENT CLIM:MEDIUM
CLIM-BACKEND:PORT-MOTION-HINTS CLIM-BACKEND:PORT-SET-MIRROR-REGION
CLIM-BACKEND:PORT-SET-MIRROR-TRANSFORMATION
CLIM-BACKEND:SELECTION-OWNER CLIM-INTERNALS::TEXT-BOUNDING-RECTANGLE*
CLIM-INTERNALS::XPM-PARSE-FILE
;   Undefined variables:
;     CLIM:*INPUT-WAIT-HANDLER* CLIM:*INPUT-WAIT-TEST*
CLIM:*POINTER-BUTTON-PRESS-HANDLER* CLIM:+EVERYWHERE+ CLIM:+NOWHERE+
;   caught 7 WARNING conditions
;   caught 35 STYLE-WARNING conditions
Unhandled SERIOUS-CONDITION of type COMPILE-FAILED-ERROR is signaled:
Lisp compilation failed



From fahree at gmail.com  Wed Feb 27 01:18:21 2013
From: fahree at gmail.com (=?ISO-8859-1?Q?Far=E9?=)
Date: Tue, 26 Feb 2013 20:18:21 -0500
Subject: [mcclim-devel] mcclim failure with asdf3
In-Reply-To: <CAN7nBXcJfKDqp0CxPh5oEbM6HRAJrahoJKcNJK52L1Hi95n6qg@mail.gmail.com>
References: <CAN7nBXcJfKDqp0CxPh5oEbM6HRAJrahoJKcNJK52L1Hi95n6qg@mail.gmail.com>
Message-ID: <CAN7nBXe33z71XhzbCV9_Skpz+Q9fozk435hXVKdQQ12enOncRQ@mail.gmail.com>

Is there anyone here to fix the issues?
To apply a patch if I fix them?

??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
Minimum wage laws are but making it illegal for less productive people
to get an honest job.


On Wed, Feb 20, 2013 at 10:43 PM, Far? <fahree at gmail.com> wrote:
> ASDF3 now checks deferred warnings by default, which notably catches
> forward references to special variables.
>
> ??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
> If the Bible proves that God exists then comic books prove the existence
> of Superman.    ? seen on #Atheism IRC
>
> ; file: /home/testgrid/quicklisp-patched2/dists/quicklisp/software/mcclim-20120909-cvs/graphics.lisp
> ; in: CLIM-EXTENSIONS:DEFINE-BITMAP-FILE-READER :XPM
> ;     (CLIM-INTERNALS::XPM-PARSE-FILE PATHNAME)
> ;
> ; caught STYLE-WARNING:
> ;   undefined function: CLIM-INTERNALS::XPM-PARSE-FILE
> ;
> ; compilation unit finished
> ;   Undefined types:
> ;     CLIM:CLIM-STREAM-PANE CLIM:STREAM-OUTPUT-HISTORY-MIXIN
> CLIM-INTERNALS::TOP-LEVEL-SHEET-PANE
> ;   Undefined functions:
> ;     (SETF CLIM-BACKEND:PORT-MOTION-HINTS) (SETF
> CLIM-BACKEND:SELECTION-OWNER) (SETF CLIM-BACKEND:SELECTION-TIMESTAMP)
> CLIM-LISP:DESCRIBE CLIM:MAKE-SPACE-REQUIREMENT CLIM:MEDIUM
> CLIM-BACKEND:PORT-MOTION-HINTS CLIM-BACKEND:PORT-SET-MIRROR-REGION
> CLIM-BACKEND:PORT-SET-MIRROR-TRANSFORMATION
> CLIM-BACKEND:SELECTION-OWNER CLIM-INTERNALS::TEXT-BOUNDING-RECTANGLE*
> CLIM-INTERNALS::XPM-PARSE-FILE
> ;   Undefined variables:
> ;     CLIM:*INPUT-WAIT-HANDLER* CLIM:*INPUT-WAIT-TEST*
> CLIM:*POINTER-BUTTON-PRESS-HANDLER* CLIM:+EVERYWHERE+ CLIM:+NOWHERE+
> ;   caught 7 WARNING conditions
> ;   caught 35 STYLE-WARNING conditions
> Unhandled SERIOUS-CONDITION of type COMPILE-FAILED-ERROR is signaled:
> Lisp compilation failed



From avodonosov at yandex.ru  Wed Feb 27 01:54:03 2013
From: avodonosov at yandex.ru (Anton Vodonosov)
Date: Wed, 27 Feb 2013 05:54:03 +0400
Subject: [mcclim-devel] mcclim failure with asdf3
Message-ID: <202211361930043@web29e.yandex.ru>

In addition, mcclim and all the dependent systems fail on CLISP and Allegro with ASDF3
because .asd file tries to define a method for asdf:traverse generic function.

The method signature is incompatible with the generic function anymore -
doesn't have &REST or &KEY.

Best regards,
- Anton




From fahree at gmail.com  Wed Feb 27 02:18:43 2013
From: fahree at gmail.com (=?ISO-8859-1?Q?Far=E9?=)
Date: Tue, 26 Feb 2013 21:18:43 -0500
Subject: [mcclim-devel] mcclim failure with asdf3
In-Reply-To: <202211361930043@web29e.yandex.ru>
References: <202211361930043@web29e.yandex.ru>
Message-ID: <CAN7nBXc24L6-goz9QD4FfwL18CN0jv8mMBCP7V0zo2b9HfMATA@mail.gmail.com>

Note that these methods on traverse that break the build on asdf3
had been totally ineffective since before asdf 2.000,
for my changes to make traverse performance linear rather than cubic
in the size of the system were such that these methods would likely
never be called.

I can provide a working replacement, but I'm not going to bother unless
there's an active committer to commit my patch.

??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
As far as we know, our computer has never had an undetected error.
                ? Weisert


On Tue, Feb 26, 2013 at 8:54 PM, Anton Vodonosov <avodonosov at yandex.ru> wrote:
> In addition, mcclim and all the dependent systems fail on CLISP and Allegro with ASDF3
> because .asd file tries to define a method for asdf:traverse generic function.
>
> The method signature is incompatible with the generic function anymore -
> doesn't have &REST or &KEY.
>
> Best regards,
> - Anton
>



From fahree at gmail.com  Wed Feb 27 08:22:51 2013
From: fahree at gmail.com (=?ISO-8859-1?Q?Far=E9?=)
Date: Wed, 27 Feb 2013 03:22:51 -0500
Subject: [mcclim-devel] mcclim failure with asdf3
In-Reply-To: <CAN7nBXc24L6-goz9QD4FfwL18CN0jv8mMBCP7V0zo2b9HfMATA@mail.gmail.com>
References: <202211361930043@web29e.yandex.ru>
	<CAN7nBXc24L6-goz9QD4FfwL18CN0jv8mMBCP7V0zo2b9HfMATA@mail.gmail.com>
Message-ID: <CAN7nBXdUjvaBjaH67UogQ0zFaOG0D90pPdf7vnF_1nS678SkwQ@mail.gmail.com>

Here is a patch to mcclim.asd that solves several issues with it.
It does *not* solve the issue with warnings;
someone who better understands McCLIM needs to address them.

What it does is:
* simplify the (require :clx) code for allegro and clisp.
 The previous traverse methods caused an error on asdf3
 because of a signature change, but were broken since asdf2 (years ago),
 because of a semantic change in traverse (necessary to make its
 complexity linear rather than cubic in the size of a system).
 That was a showstopper. Instead, I do things in much simpler ways,
 that work and are supported.

* remove the braindead perform :around binding *features*,
 that totally defeated the other perform :after methods that modified it.
 Instead, don't bind *features*, but just push :mcclim on the features
 earlier, early enough to compile ESA and Drei with it
 (and yes, grepping through the code shows it works; the :building-mcclim
 feature is always redundant with :mcclim; it should just die.)

* all these uses of make-pathname were necessary in ASDF1,
 but haven't been since ASDF2. I simplified them away.

One of you still has to resolve all the warning issues.
Some of them *might* disappear if you merge some of your systems together.
Or not.

??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
La vie n'est pas un restaurant mais un buffet. Levez-vous pour vous servir.
	Life is not a restaurant but a buffet. Stand up to serve yourself.
		? Dominique Glocheux


On Tue, Feb 26, 2013 at 9:18 PM, Far? <fahree at gmail.com> wrote:
> Note that these methods on traverse that break the build on asdf3
> had been totally ineffective since before asdf 2.000,
> for my changes to make traverse performance linear rather than cubic
> in the size of the system were such that these methods would likely
> never be called.
>
> I can provide a working replacement, but I'm not going to bother unless
> there's an active committer to commit my patch.
>
> ??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
> As far as we know, our computer has never had an undetected error.
>                 ? Weisert
>
>
> On Tue, Feb 26, 2013 at 8:54 PM, Anton Vodonosov <avodonosov at yandex.ru> wrote:
>> In addition, mcclim and all the dependent systems fail on CLISP and Allegro with ASDF3
>> because .asd file tries to define a method for asdf:traverse generic function.
>>
>> The method signature is incompatible with the generic function anymore -
>> doesn't have &REST or &KEY.
>>
>> Best regards,
>> - Anton
>>



From fahree at gmail.com  Wed Feb 27 15:30:37 2013
From: fahree at gmail.com (=?ISO-8859-1?Q?Far=E9?=)
Date: Wed, 27 Feb 2013 10:30:37 -0500
Subject: [mcclim-devel] mcclim failure with asdf3
In-Reply-To: <CAN7nBXdUjvaBjaH67UogQ0zFaOG0D90pPdf7vnF_1nS678SkwQ@mail.gmail.com>
References: <202211361930043@web29e.yandex.ru>
	<CAN7nBXc24L6-goz9QD4FfwL18CN0jv8mMBCP7V0zo2b9HfMATA@mail.gmail.com>
	<CAN7nBXdUjvaBjaH67UogQ0zFaOG0D90pPdf7vnF_1nS678SkwQ@mail.gmail.com>
Message-ID: <CAN7nBXeUSaGjGZtUzPyg_u545+BBt5sebPdG9F2k0Jg_JftaGQ@mail.gmail.com>

Works better with the patch attached.

Once again, this only fixes the build on clisp and allegro,
and solves a fiasco with *features*, but it doesn't address
the many warnings in the build, that some developer should address.

??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
To have a right to do a thing is not at all the same
as to be right in doing it.     ? G.K. Chesterton


On Wed, Feb 27, 2013 at 3:22 AM, Far? <fahree at gmail.com> wrote:
> Here is a patch to mcclim.asd that solves several issues with it.
> It does *not* solve the issue with warnings;
> someone who better understands McCLIM needs to address them.
>
> What it does is:
> * simplify the (require :clx) code for allegro and clisp.
>  The previous traverse methods caused an error on asdf3
>  because of a signature change, but were broken since asdf2 (years ago),
>  because of a semantic change in traverse (necessary to make its
>  complexity linear rather than cubic in the size of a system).
>  That was a showstopper. Instead, I do things in much simpler ways,
>  that work and are supported.
>
> * remove the braindead perform :around binding *features*,
>  that totally defeated the other perform :after methods that modified it.
>  Instead, don't bind *features*, but just push :mcclim on the features
>  earlier, early enough to compile ESA and Drei with it
>  (and yes, grepping through the code shows it works; the :building-mcclim
>  feature is always redundant with :mcclim; it should just die.)
>
> * all these uses of make-pathname were necessary in ASDF1,
>  but haven't been since ASDF2. I simplified them away.
>
> One of you still has to resolve all the warning issues.
> Some of them *might* disappear if you merge some of your systems together.
> Or not.
>
> ??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
> La vie n'est pas un restaurant mais un buffet. Levez-vous pour vous servir.
>         Life is not a restaurant but a buffet. Stand up to serve yourself.
>                 ? Dominique Glocheux
>
>
> On Tue, Feb 26, 2013 at 9:18 PM, Far? <fahree at gmail.com> wrote:
>> Note that these methods on traverse that break the build on asdf3
>> had been totally ineffective since before asdf 2.000,
>> for my changes to make traverse performance linear rather than cubic
>> in the size of the system were such that these methods would likely
>> never be called.
>>
>> I can provide a working replacement, but I'm not going to bother unless
>> there's an active committer to commit my patch.
>>
>> ??? ? Fran?ois-Ren? ?VB Rideau ?Reflection&Cybernethics? http://fare.tunes.org
>> As far as we know, our computer has never had an undetected error.
>>                 ? Weisert
>>
>>
>> On Tue, Feb 26, 2013 at 8:54 PM, Anton Vodonosov <avodonosov at yandex.ru> wrote:
>>> In addition, mcclim and all the dependent systems fail on CLISP and Allegro with ASDF3
>>> because .asd file tries to define a method for asdf:traverse generic function.
>>>
>>> The method signature is incompatible with the generic function anymore -
>>> doesn't have &REST or &KEY.
>>>
>>> Best regards,
>>> - Anton
>>>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: asdf2.diff
Type: application/octet-stream
Size: 13991 bytes
Desc: not available
URL: <https://mailman.common-lisp.net/pipermail/mcclim-devel/attachments/20130227/f44cbe5a/attachment.obj>