[ragel-users] [PATCH 0/3] Fix Cygwin build

[Ken Brown]

Hi Adrian,

Thanks.  Have you pushed these changes to the upstream git repo?  I'm getting 
"Already up to date" when I do "git pull".

Ken

On 11/25/2018 8:22 AM, Adrian Thurston wrote:
> Hi Ken, thank you for the patches. I just enabled --no-undefined and removed the 
> reference to the undefined symbol. The parse.c file cannot be included in 
> libragel because it defines the frontnend language, which is different for each 
> binary in the host-* directories. All that was needed was to pass the reference 
> to rl_parse into libragel from main.cc. Was already doing that for the backend 
> code generator (also a colm program).
> 
> Adrian
> 
> On 2018-11-16 16:58, Ken Brown wrote:
>> I've just built ragel-7.0.0.11 on Cygwin and added it to the Cygwin
>> distribution.  I had to patch the sources in order to build shared
>> libraries.
>>
>> libtool won't build shared libraries on Cygwin unless the
>> -no-undefined flag is given.  The first patch of this series does that
>> for libfsm and libragel.  But it turns out that there are in fact
>> undefined symbols in libragel.  The next two patches fix that.
>>
>> I don't think these patches do any harm on other platforms.
>>
>> Ken Brown (3):
>>   add -no-undefined to LDFLAGS for libfsm and libragel
>>   add dependency of libragel on libfsm and libcolm
>>   avoid undefined symbols in libragel
>>
>>  src/Makefile.am | 20 ++++++++++++++------
>>  1 file changed, 14 insertions(+), 6 deletions(-)
> 
> _______________________________________________
> ragel-users mailing list
> ragel-users at colm.net
> http://www.colm.net/cgi-bin/mailman/listinfo/ragel-users