Issue with GAP_MakeStringWithLen and MarkBag for GAP v4.13.0 #22
Description
Hi There
I just tried to install gappy on top of gap v4.13 and it seems that there are some issues with MarkBag and GAP_MakeStringWithLen.
gcc -pthread -B /opt/conda/compiler_compat -DNDEBUG -fwrapv -O2 -Wall -fPIC -O2 -isystem /opt/conda/include -fPIC -O2 -isystem /opt/conda/include -fPIC -DCYTHON_USE_PYLONG_INTERNALS=0 -I/tmp/pip-build-env-1hqozfcp/overlay/lib/python3.11/site-packages/cysignals -Ibuild/temp.linux-x86_64-cpython-311/include -I/usr/local/src/gap-4.13.0/gen -I/opt/conda/include -I/opt/conda/include/python3.11 -c gappy/core.c -o build/temp.linux-x86_64-cpython-311/gappy/core.o
8.558 gappy/core.c:816:23: error: conflicting types for ‘GAP_MakeStringWithLen’; have ‘struct OpaqueBag *(char *, size_t)’ {aka ‘struct OpaqueBag *(char *, long unsigned int)’}
8.558 816 | static inline Obj GAP_MakeStringWithLen(char *s, size_t len) {
8.558 | ^~~~~~~~~~~~~~~~~~~~~
8.558 In file included from gappy/core.c:782:
8.558 build/temp.linux-x86_64-cpython-311/include/gap/libgap-api.h:553:5: note: previous declaration of ‘GAP_MakeStringWithLen’ with type ‘struct OpaqueBag *(const char *, UInt)’ {aka ‘struct OpaqueBag *(const char *, long unsigned int)’}
8.558 553 | Obj GAP_MakeStringWithLen(const char * string, UInt len);
8.558 | ^~~~~~~~~~~~~~~~~~~~~
8.558 gappy/core.c: In function ‘__pyx_f_5gappy_4core_gasman_callback’:
8.558 gappy/core.c:3857:5: error: too few arguments to function ‘MarkBag’
8.558 3857 | MarkBag(((struct __pyx_obj_5gappy_4core_ObjWrapper *)__pyx_v_obj)->value);
8.558 | ^~~~~~~
8.558 In file included from gappy/core.c:809:
8.558 build/temp.linux-x86_64-cpython-311/include/gap/gasman.h:758:6: note: declared here
8.558 758 | void MarkBag(Bag bag, void * ref);
8.558 | ^~~~~~~
8.558 error: command '/usr/bin/gcc' failed with exit code 1
8.558 [end of output]
I am at a complete loss here, but it looks like you were aware of the coming issue with: GAP_MakeStringWithLen. The MarkBag Issue might be a bit more involved, since the number of parameters has been extended.
Any chance you could have a look? That would be greatly appreciated!