All of lore.kernel.org
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: Pascal Bach <pascal.bach@siemens.com>
Cc: "yocto@yoctoproject.org" <yocto@yoctoproject.org>
Subject: Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
Date: Tue, 7 Nov 2017 08:36:41 -0800	[thread overview]
Message-ID: <CAMKF1spLYuAWAXS_MnkJhEC3Kc1VZ8s4saixa29fVOTXYiDkoA@mail.gmail.com> (raw)
In-Reply-To: <1bd6b35a-c58e-df9c-2855-e2c3657ea360@siemens.com>

On Tue, Nov 7, 2017 at 5:26 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
>
>
>>>>
>>>> Mono requires cmake to build the bundled BoringSSL library. I didn't look why it works in detail but I currently assume that BoringSSL is standalone enough that no additional
>>>> CMake toolchain settings are required to make it work.
>>>>
>>> yeah that probably is a case of bundling which is on rise at app level
>>> these days,
>>>
>>>> One option would be to disable it with `--disable-btls` but I fear that would disable TLS1.2 support (http://www.mono-project.com/docs/about-mono/releases/4.8.0/#tls-12-support).
>>>> But I have no way of checking that.
>>>>
>>> interesting, do they support any other TLS implementations ?
>>>
>>
>> It is disabled in 5.x builds by default as it breaks the build e.g.
>>
>> http://git.yoctoproject.org/cgit/cgit.cgi/meta-mono/tree/recipes-mono/mono/mono-5.4.0.201.inc
>>
>> It's probably time to look at a working build with it re-enabled. Patches welcomed...
>>
>> (I did have a quick look at inheriting cmake but that breaks the build too)
>>
> I did some more testing. And adding "cmake-native" still doesn't build btls. I guess it need cmake to figure that out. But this means it was never enabled in the first place.
>
> So my proposal for the moment is to explicitly set "--disable-btls" for Mono 4 too, this way it builds and it is clear that it is disabled.

that seems reasonable

>
> If somebody figures out how to enable it again. I will send an updated Patch.
>
> Pascal
>


      parent reply	other threads:[~2017-11-07 16:37 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-27 10:51 [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake Pascal Bach
2017-10-27 21:41 ` Khem Raj
2017-11-02 15:46   ` Pascal Bach
2017-11-02 15:59     ` Khem Raj
2017-11-03 11:48       ` Alex Lennon
2017-11-07 13:26         ` Pascal Bach
2017-11-07 13:57           ` Alex Lennon
2017-11-07 16:36           ` Khem Raj [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAMKF1spLYuAWAXS_MnkJhEC3Kc1VZ8s4saixa29fVOTXYiDkoA@mail.gmail.com \
    --to=raj.khem@gmail.com \
    --cc=pascal.bach@siemens.com \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.