Re: libgst/Makefile.am diff

[Holger Freyther]

Hi Jean-Marc,

Thank you for the patch and sorry for the delay. Any chance that you could try
the version that I just sent to the mailinglist?

On my debian/aarch64 bookwork build I do see some strange errors in the 
generated
match.h. It seems like it dropped the bytecode names. Have you seen this happen
as well?

match.h dropping names...

#define MATCH_BYTECODES_ \
  MATCH_BYTECODES__: \
   \
  goto MATCH_BYTECODES_SWITCH_; \
  MATCH_BYTECODES__: \

> On 8. Mar 2024, at 02:49, Holger Freyther <holger@freyther.de> wrote:
> 
> Thank you! Let me AX_CHECK_LINK_FLAG and land this quickly!
> 
>> On 26. Feb 2024, at 13:10, Jean-Marc Farinas <jmfarinas31@gmail.com> wrote:
>> 
>> I've finally found a way (quite accidentally) to add linker flags when 
>> compiling genbc. Modifing grammar file did not work for me either (to heavy 
>> code modification without proper understanding of how the vm is built).  And 
>> requesting a linker to accept having multiple definition is acceptable since 
>> genbc impl and decl parsers dont work at the same time.
>> 
>> diff --git a/libgst/Makefile.am b/libgst/Makefile.am
>> index 8841ea4c..82198e8f 100644
>> --- a/libgst/Makefile.am
>> +++ b/libgst/Makefile.am
>> @@ -50,7 +50,8 @@ genprims_LDADD = @LIBSNPRINTFV@ 
>> $(top_builddir)/lib-src/library.la
>> genbc_SOURCES = \
>>        genbc-decl.y genbc-impl.y genbc-scan.l genbc.c
>> 
>> -genbc_LDADD = @LIBSNPRINTFV@ $(top_builddir)/lib-src/library.la
>> +genbc_LDADD =  @LIBSNPRINTFV@ $(top_builddir)/lib-src/library.la
>> +genbc_LDFLAGS = -Wl,-allow-multiple-definition
>> 
>> # definitions for genvm
>> 
>> 
> 
>