All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Lennon <ajlennon@dynamicdevices.co.uk>
To: Pascal Bach <pascal.bach@siemens.com>, Khem Raj <raj.khem@gmail.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 13:57:09 +0000	[thread overview]
Message-ID: <a5f3d58a-c0d2-6c26-81a5-c544679b2f5f@dynamicdevices.co.uk> (raw)
In-Reply-To: <1bd6b35a-c58e-df9c-2855-e2c3657ea360@siemens.com>



On 07/11/2017 13:26, Pascal Bach 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.
>
> If somebody figures out how to enable it again. I will send an updated Patch.
>
> Pascal
>
>

Pascal,

For now I've put a couple of commits into master which re-enable BTLS 
for 5.8.022 and 5.4.0.201. Both of these seem to build with your cmake 
dependency added.

After listening to the team at #OEDEM I've been working to add basic 
Image Test support to core-image-mono and to get the Yocto Auto Builder 
going so I can automate some testing of the yocto release vs mono 
version vs mono features matrix

That's finally coming together at which point I'm hoping to genericise a 
bit more.

There are currently just three basic run time tests which previous to 
the image test support I've been performing manually

ref: 
http://git.yoctoproject.org/cgit/cgit.cgi/meta-mono/tree/lib/oeqa/runtime/cases/mono.py

If you have suggestions/patches for an image test for TLS (or indeed any 
other tests) they'll be gratefully received.

At some point I want to extend this with ptest support for running the 
Mono test suite on the target { qemu / real hardware etc }

ref: http://www.mono-project.com/community/contributing/test-suite/

Cheers,

Alex



  reply	other threads:[~2017-11-07 13:57 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 [this message]
2017-11-07 16:36           ` Khem Raj

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=a5f3d58a-c0d2-6c26-81a5-c544679b2f5f@dynamicdevices.co.uk \
    --to=ajlennon@dynamicdevices.co.uk \
    --cc=pascal.bach@siemens.com \
    --cc=raj.khem@gmail.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.