Hello,

> >> I suppose that these fields are unnecessary as Guile-Fluidsynth
> >> does not use C compiler or libtool.  
> > I see no *.c files in guile-fluidsynth, so that sounds right.  Why
> > was the patch added to debian/patches/ in the first place?  

> ChatGPT suggested it. I'll remove the configure.ac patch (0002)
> entirely as it is not needed.

Those lines are commented, I don't see the need to remove them, as I
keep them if I need a C lib in the future.

> > OTOH I think the f-synth/ffi/*.scm files should be re-generated from
> > source code using 'guile compile-ffi' during build-time.
> > Preferably the generated files shouldn't even be in git.  To make
> > sure we can rebuild everything from source code.

This needs a more complicated autotool chain setting, and imo, it's
unnecessary, as all generated upstream will always work fine 'for all'
- is this really a problem?

David

Attachment: pgpqIKQvB9WVt.pgp
Description: OpenPGP digital signature

Reply via email to