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

[Adrian Thurston]

Ah, sorry, I forgot to push from my private repo to the public one. 
There now.

On 2018-11-25 12:11, Ken Brown wrote:
> 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
> _______________________________________________
> ragel-users mailing list
> ragel-users at colm.net
> http://www.colm.net/cgi-bin/mailman/listinfo/ragel-users