Hi Ellie,

> When you configure it with your patch applied and 1.1.0 installed,
does Cyrus build okay?

Yes, it builds without errors.

Configure prints:

checking for WSLAY... yes
   wslay:              yes

And -lwslay is passed as arg numerous times during build process.

And effectively httpd binary includes references to wslay_event_xxx in
its symbols table.

Regards,
Anatoli

On 3/6/20 01:35, ellie timoney wrote:
> In our "cyruslibs" package, the wslay submodule is at this commit:
> 
> commit 4a937cd (HEAD, origin/master, origin/HEAD, master)
> Author:     Tatsuhiro Tsujikawa <tatsuhir...@gmail.com>
> AuthorDate: Fri Jun 8 23:19:03 2018 +0900
> Commit:     Tatsuhiro Tsujikawa <tatsuhir...@gmail.com>
> CommitDate: Fri Jun 8 23:19:03 2018 +0900
> 
>     Bump up version number to 1.1.1-DEV
> 
> Which is the commit immediately following the release-1.1.0 tag.  So, 
> presumably, we're not dependent on any feature/fix that's only in the 
> unreleased version, because otherwise we would've bumped the cyruslibs 
> submodule to include those commits?
> 
> So, "1.1.1" might be a typo, or an anticipatory thing that didn't go anywhere 
> yet, I'm not sure.
> 
> When you configure it with your patch applied and 1.1.0 installed, does Cyrus 
> build okay?
> 
> Cheers,
> 
> ellie
> 
> On Wed, Jun 3, 2020, at 2:13 PM, Anatoli wrote:
>> Cyrus developers,
>>
>> The configure script checks for wslay lib version 1.1.1, but the latest 
>> version released is 1.1.0. So when it is installed, it reports:
>>
>> checking for WSLAY... no
>> configure: httpd will not have support for WebSockets.  Consider 
>> installing libwslay
>>
>> The wslay's github repo has a mention of a 1.1.1-DEV version. Not sure 
>> if cyrus-imapd httpd requires something from it or if it was just a 
>> typo and 1.1.0 is ok.
>>
>> For the later case below is a patch.
>>
>> Regards,
>> Anatoli
>>
>>
>> diff --git a/configure.ac b/configure.ac
>> index dc0e0fff2..30e925c60 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1631,7 +1631,7 @@ dnl                AC_MSG_WARN([Your version of 
>> OpenDKIM can not support iSchedu
>>
>>          AC_ARG_WITH(wslay, [AS_HELP_STRING([--without-wslay], [disable 
>> WebSockets support (check)])],,[with_wslay="check"])
>>          if test "x$with_wslay" = "xyes" -o "x$with_wslay" = "xcheck"; 
>> then
>> -                PKG_CHECK_MODULES([WSLAY], [libwslay >= 1.1.1], [
>> +                PKG_CHECK_MODULES([WSLAY], [libwslay >= 1.1.0], [
>>                          AC_DEFINE(HAVE_WSLAY,[],
>>                                  [Build WebSockets support into httpd?])
>>                          with_wslay=yes
>>

Reply via email to