Hello David.
David Pirotte writes:
> If you could kindly grab the latest, run the make/install dance on your
> respective distro and try again ... Here the example now runs for
> hours,
Yes, same on Guix. It runs for more than an hour.
> as it should, consuming +/- 3% cpu and a steady 0.8% of
David Pirotte writes:
> Hello Florian,
> Matija,
>
>> > > all of a sudden it stops and raises an exception - could you also
>> > > try that and let me know, thanks
>
>> > Yes, after 5 minutes animated-paintable.scm crashes. Hmm.
>
>> Ok, thanks for trying - This is going to be quite a lot
> ...
> Here the example now runs for hours, as it should, consuming +/- 3%
> cpu and a steady 0.8% of mem (so afaict, no memory leak either).
Actually the example memory consumption grows, very slowly but it grows.
I think I know why, I'll try to fix this as well, but I need to finish
some other
Hello Florian,
Matija,
> > > all of a sudden it stops and raises an exception - could you also
> > > try that and let me know, thanks
> > Yes, after 5 minutes animated-paintable.scm crashes. Hmm.
> Ok, thanks for trying - This is going to be quite a lot more complex
> to debug and fix, an
Hi Florian,
> Yes, they had not worked previously, and now with your long-name fix,
> they work.
Fwiw, they actually worked fine till day one till 2024-09-04, the day i
pushed this commit a3952000ae205c27573ab0eaf2176131b51e34cf,
that 'broke' them ...
But i am glad you confirm everything is fine
David Pirotte writes:
> Could you both kindly grab the latest devel branch, you need this
> commit 5d72bfcdae85bb4a5b0254dc9f22fb8c676db69d, and try
> both the animated-paintable.scm and peg-solitaire.scm ex. ... let me
> know.
animated-paintable and peg-solitaire I had not tried last time and
th
Matija,
Florian,
> > Except for "peg-solitaire.scm" and "animated-paintable.scm" all
> > examples works fine, thank you.
> I can reproduce the bug here, let me investigate, i'll get back asap
I think i fixed that (recently introduced) bug [1].
Could you both kindly grab the latest devel branc
Hi Matija,
Florian,
This example should 'now' likely fail in guix as well (?)
> Except for "peg-solitaire.scm" and "animated-paintable.scm" all
> examples works fine, thank you.
I can reproduce the bug here, let me investigate, i'll get back asap ...
Thanks for the report,
David
Hi again,
> > P.S. on Guix, I also no longer have to use --no-grafts, possibly
> > because Guix had major updates to core packages.
> Excellent!
> > G-golf really satisfies now.
> At last! Note that the above was not a g-golf problem [1].
Just want to make sure we are (all) on the same wave
> P.S. on Guix, I also no longer have to use --no-grafts, possibly
> because Guix had major updates to core packages.
Excellent!
> G-golf really satisfies now.
At last! Note that the above was not a g-golf problem [1].
Regards,
David
[1]
And for that matter, nor is (a g-golf problem) the pat
Hi Florian,
> Thank you David, thank you Matija,
> it works on Guix!
Excellent!
> Except for small warnings like
> Warning: Missing /etc/X11/rgb.txt, using a fallback.
> that probably should be fixed on the Guix side.
You may as well safely ignore the warning ofc - i am not sure how to
handle t
Hello David,
> Could you both kindly grab the latest devel branch, you need this
> commit bbfee04922bf2a22db397e2f429ae606df940265, and run the
> make/install danse on your respective distro, then check if the
> 'problematic' examples work fine _even when_ run against a modern libffi
> that was co
P.S. on Guix, I also no longer have to use --no-grafts, possibly because
Guix had major updates to core packages. G-golf really satisfies now.
Also thank you to the GJS maintainer for their help.
Regards,
Florian
Thank you David, thank you Matija,
it works on Guix!
Except for small warnings like
Warning: Missing /etc/X11/rgb.txt, using a fallback.
that probably should be fixed on the Guix side.
And some issues that I have to chdir to
~/src/g-golf/examples/gtk-4/
to run
~/src/g-golf/examples/gtk-4/revealer
Hello Matija,
Florian,
> > > I still wonder where is the problem, in Guile, G-Golf or
> > > gobject-introspection (which according to [2] should not be a
> > > problem).
> > Definitely not a g-golf bug.
Further speaking with other GI lang binding authors, the gjs author
finally did point m
Hi again,
> ...
> > I still wonder where is the problem, in Guile, G-Golf or
> > gobject-introspection (which according to [2] should not be a
> > problem).
> Definitely not a g-golf bug.
Unless the code used by g-golf to make the ffi-cif would need to be
adapted, i don't know, but one thing
Hello Matija,
> This flag was recently removed [2] in nixpkgs with a comment:
> > The fix for gobject-introspection is in version 1.72.
Maybe the fix didn't really fixed the problem ...
> Bottom line: If I compile Guile with patched libffi [3] dependency
> (where flag "disable-exec-static-tram
Hello Matija. I confirm that on Guix as well static trampolines are the
issue; using
((package-input-rewriting
`((,libffi . ,libffi-sans-static-trampolines)))
guile-3.0)
as Guile to run g-golf demos makes every g-golf demo work without patches.
Regards,
Florian
Hello David,
> So your patch seems to point to a problem in the upstream definition of
> the (most recent) g-callable-info-create-closure, which is the one
> everybody who's using GI >= 1.71.0 should use ...
I see. The comment in deprecated function made me thinking and I found
notice in LFS [1]:
Hello Matija,
> There is a version check in "g-callable-info-make-closure" function,
> if I disable it (check patch) all examples works just fine.
Great! But ...
> I'm not quite sure what these functions does, can you take another
> look?
g-callable-info-make-closure just select in between the
Wow, Matija, good idea; this work-around fixes all examples on Guix, too
(using gobject-introspection version 1.78.1).
Regards,
Florian
Hello again,
I think I found something.
There is a version check in "g-callable-info-make-closure" function, if
I disable it (check patch) all examples works just fine.
I'm not quite sure what these functions does, can you take another look?
My gi version is 1.80.1.
>From 4bec7836f17dfd453b877
"pelzflorian (Florian Pelz)" writes:
>>> The feedback I can give: guile-zlib in its build system autodetects
>>> ...
>> Not sure i want to do this.
>
> Yeah, normally there is no need, when using the package.
>
> I do not know what Matija did on Nix, because I cannot find any g-golf
> in Nixpkgs.
Hello David.
David Pirotte writes:
> Thanks for the nice words - I hope that some day, with the help of
> Guix/Nix maintainer(s), we can identify and solve the problem.
The other Nix patches for giscanner had been written by Luca Bruno who
did work on gobject-introspection itself, although Guix
Hi Florian,
> Hello David. Thank you for this important groundwork that g-golf is.
Thanks for the nice words - I hope that some day, with the help of
Guix/Nix maintainer(s), we can identify and solve the problem.
> Sorry to say, I have not developed any GUI and had no time as planned.
> It will
Hello David. Thank you for this important groundwork that g-golf is.
Sorry to say, I have not developed any GUI and had no time as planned.
It will not happen with me. But Debian is not the problem; all is fine
there (unlike G-Golf on Guix on Debian).
I also failed to put a libg-golf-tests libr
Hi Florian,
is your debian host g-golf experience ok? i know it is a
temporary solution, till we find a solution for nix/guix, but
i am curious - also, is the app you are working on free s/w,
any public repo we could look at?
> Hello, Yes, the same issue still pla
Hello Matija,
> I'm facing the problem mentioned in [1]. Problematic are all examples
> with virtual methods.
Ok, so you are facing the same problem as Guix and HomeBrew, all other
distro (that tried g-golf) work fine, i use Debian.
> I have tried simpler example (attached) with same result. The
Hello, Yes, the same issue still plagues me. Never tried guile-gi and
I do not know the difference regarding vfuncs. Is it like pygobject
that guile-gi does not use Guile as much and more C?
My only progress:
> I'm using Nix.
Interesting. Guix and Nix have their own settings to load librarie
29 matches
Mail list logo
