New CFFI fails to load cl-cairo2-xlib
Latest head 9b312d0d7e38 (ql:quickload :cl-cairo2-xlib) Many messages like ; caught ERROR: ; don't know how to dump #<CFFI::AGGREGATE-STRUCT-SLOT {1016815063}> (default MAKE-LOAD-FORM method called). ; ==> ; #<CFFI::AGGREGATE-STRUCT-SLOT {1016815693}> eventually abort to ; ; compilation unit aborted ; caught 2 fatal ERROR conditions ; caught 54 ERROR conditions (:CL-CAIRO2-XLIB) When rolled back to 26f11a2bfe6c, everything is fine. On SBCL, amd64 Debian. Liam
the error is in the struct accessors, they end up containing literal FOREIGN-STRUCT-SLOT objects without a MAKE-LOAD-FORM. i've added a test that exhibits this problem: https://github.com/cffi/cffi/commit/4310a7a7d774af06a63df3a30183293ba08756f1 the fix is not trivial, i gave up. i tried to make it expand into a form that looks up the struct and the slot at load-time (in a MAKE-LOAD-FORM), but the slots don't know about their structs, so i cannot walk up and then back down the hierarchy. -- • attila lendvai • PGP: 963F 5D5F 45C7 DFCD 0A39 -- Violation of consent is the foundation of all crime. Corollary 1: No victim? Then no crime. Corollary 2: The State is the most pervasive criminal gang known to man.
i've added a test that exhibits this problem:
and i forgot about travis again... will someone be able to come up with a fix soonish, or shall i'll squeeze that into a compile-file? -- • attila lendvai • PGP: 963F 5D5F 45C7 DFCD 0A39 -- “The pursuit of commerce reconciles nations, calms wars, strengthens peace, and commutes the private good of individuals into the common benefit of all.” — Hugh of Saint Victor (1096–1141)
On Tue, Feb 2, 2016 at 11:32 AM, Attila Lendvai <attila@lendvai.name> wrote:
and i forgot about travis again... will someone be able to come up with a fix soonish, or shall i'll squeeze that into a compile-file?
No problem. :) I'll try to fix it over lunch (that's in a couple of hours time). Cheers, -- Luís Oliveira http://kerno.org/~luis/
On Tue, Feb 2, 2016 at 11:35 AM, Luís Oliveira <luismbo@gmail.com> wrote:
On Tue, Feb 2, 2016 at 11:32 AM, Attila Lendvai <attila@lendvai.name> wrote:
and i forgot about travis again... will someone be able to come up with a fix soonish, or shall i'll squeeze that into a compile-file?
No problem. :) I'll try to fix it over lunch (that's in a couple of hours time).
Should be fixed now. Liam, can you confirm your problem was fixed? Cheers, -- Luís Oliveira http://kerno.org/~luis/
Yes, loads fine with a74ecd7b6492. Thanks. Liam On Tue, Feb 2, 2016 at 6:22 PM, Luís Oliveira <luismbo@gmail.com> wrote:
On Tue, Feb 2, 2016 at 11:35 AM, Luís Oliveira <luismbo@gmail.com> wrote:
On Tue, Feb 2, 2016 at 11:32 AM, Attila Lendvai <attila@lendvai.name> wrote:
and i forgot about travis again... will someone be able to come up with a fix soonish, or shall i'll squeeze that into a compile-file?
No problem. :) I'll try to fix it over lunch (that's in a couple of hours time).
Should be fixed now. Liam, can you confirm your problem was fixed?
Cheers,
-- Luís Oliveira http://kerno.org/~luis/
participants (3)
-
Attila Lendvai -
Liam Healy -
Luís Oliveira