Re: libgst/Makefile.am diff

Holger Freyther <[email protected]> Sat, 16 Mar 2024 14:39:14 +0800
Newsgroups gmane.comp.lang.smalltalk.gnu.general
Message-ID <[email protected]>
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 <[email protected]> wrote:
>=20
> Thank you! Let me AX_CHECK_LINK_FLAG and land this quickly!
>=20
>> On 26. Feb 2024, at 13:10, Jean-Marc Farinas <[email protected]> =
wrote:
>>=20
>> 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.
>>=20
>> 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 =3D @LIBSNPRINTFV@ =
$(top_builddir)/lib-src/library.la
>> genbc_SOURCES =3D \
>>        genbc-decl.y genbc-impl.y genbc-scan.l genbc.c
>>=20
>> -genbc_LDADD =3D @LIBSNPRINTFV@ $(top_builddir)/lib-src/library.la
>> +genbc_LDADD =3D  @LIBSNPRINTFV@ $(top_builddir)/lib-src/library.la
>> +genbc_LDFLAGS =3D -Wl,-allow-multiple-definition
>>=20
>> # definitions for genvm
>>=20
>>=20
>=20
>=20