[meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Pascal Bach]

This was probably working before pyro and the recipes specific sysroots.

Signed-off-by: Pascal Bach <pascal.bach@siemens.com>
---
 recipes-mono/mono/mono-4.xx.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recipes-mono/mono/mono-4.xx.inc b/recipes-mono/mono/mono-4.xx.inc
index e06fac7..cf9170b 100644
--- a/recipes-mono/mono/mono-4.xx.inc
+++ b/recipes-mono/mono/mono-4.xx.inc
@@ -4,7 +4,7 @@ HOMEPAGE = "http://mono-project.com"
 BUGTRACKER = "http://bugzilla.xamarin.com/"
 SECTION = "devel"
 
-DEPENDS = "zlib"
+DEPENDS = "zlib cmake-native"
 
 SRC_URI = "http://download.mono-project.com/sources/mono/mono-${PV}.tar.bz2 \
            file://dllmap-config.in.diff \
-- 
2.1.4

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Khem Raj]

On Fri, Oct 27, 2017 at 3:51 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
> This was probably working before pyro and the recipes specific sysroots.
>
> Signed-off-by: Pascal Bach <pascal.bach@siemens.com>
> ---
>  recipes-mono/mono/mono-4.xx.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/recipes-mono/mono/mono-4.xx.inc b/recipes-mono/mono/mono-4.xx.inc
> index e06fac7..cf9170b 100644
> --- a/recipes-mono/mono/mono-4.xx.inc
> +++ b/recipes-mono/mono/mono-4.xx.inc
> @@ -4,7 +4,7 @@ HOMEPAGE = "http://mono-project.com"
>  BUGTRACKER = "http://bugzilla.xamarin.com/"
>  SECTION = "devel"
>
> -DEPENDS = "zlib"
> +DEPENDS = "zlib cmake-native"
>

perhaps this should inehrit cmake if its using cmake
or explain how cmake is used where abstactions
are not relevant


>  SRC_URI = "http://download.mono-project.com/sources/mono/mono-${PV}.tar.bz2 \
>             file://dllmap-config.in.diff \
> --
> 2.1.4
>
> --
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Pascal Bach]

On 27.10.2017 23:41, Khem Raj wrote:
> On Fri, Oct 27, 2017 at 3:51 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
>> This was probably working before pyro and the recipes specific sysroots.
>>
>> Signed-off-by: Pascal Bach <pascal.bach@siemens.com>
>> ---
>>  recipes-mono/mono/mono-4.xx.inc | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/recipes-mono/mono/mono-4.xx.inc b/recipes-mono/mono/mono-4.xx.inc
>> index e06fac7..cf9170b 100644
>> --- a/recipes-mono/mono/mono-4.xx.inc
>> +++ b/recipes-mono/mono/mono-4.xx.inc
>> @@ -4,7 +4,7 @@ HOMEPAGE = "http://mono-project.com"
>>  BUGTRACKER = "http://bugzilla.xamarin.com/"
>>  SECTION = "devel"
>>
>> -DEPENDS = "zlib"
>> +DEPENDS = "zlib cmake-native"
>>
> perhaps this should inehrit cmake if its using cmake
> or explain how cmake is used where abstactions
> are not relevant
Using the cmake bbclass doesn't work as it is not really a CMake project.

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.

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.

Pascal


>
>>  SRC_URI = "http://download.mono-project.com/sources/mono/mono-${PV}.tar.bz2 \
>>             file://dllmap-config.in.diff \
>> --
>> 2.1.4
>>
>> --
>> _______________________________________________
>> yocto mailing list
>> yocto@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/yocto

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Khem Raj]

On Thu, Nov 2, 2017 at 8:46 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
>
> On 27.10.2017 23:41, Khem Raj wrote:
>> On Fri, Oct 27, 2017 at 3:51 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
>>> This was probably working before pyro and the recipes specific sysroots.
>>>
>>> Signed-off-by: Pascal Bach <pascal.bach@siemens.com>
>>> ---
>>>  recipes-mono/mono/mono-4.xx.inc | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/recipes-mono/mono/mono-4.xx.inc b/recipes-mono/mono/mono-4.xx.inc
>>> index e06fac7..cf9170b 100644
>>> --- a/recipes-mono/mono/mono-4.xx.inc
>>> +++ b/recipes-mono/mono/mono-4.xx.inc
>>> @@ -4,7 +4,7 @@ HOMEPAGE = "http://mono-project.com"
>>>  BUGTRACKER = "http://bugzilla.xamarin.com/"
>>>  SECTION = "devel"
>>>
>>> -DEPENDS = "zlib"
>>> +DEPENDS = "zlib cmake-native"
>>>
>> perhaps this should inehrit cmake if its using cmake
>> or explain how cmake is used where abstactions
>> are not relevant
> Using the cmake bbclass doesn't work as it is not really a CMake project.
>
> 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 ?

> Pascal
>
>
>>
>>>  SRC_URI = "http://download.mono-project.com/sources/mono/mono-${PV}.tar.bz2 \
>>>             file://dllmap-config.in.diff \
>>> --
>>> 2.1.4
>>>
>>> --
>>> _______________________________________________
>>> yocto mailing list
>>> yocto@yoctoproject.org
>>> https://lists.yoctoproject.org/listinfo/yocto
>

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Alex Lennon]

On 02/11/2017 15:59, Khem Raj wrote:
> On Thu, Nov 2, 2017 at 8:46 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
>> On 27.10.2017 23:41, Khem Raj wrote:
>>> On Fri, Oct 27, 2017 at 3:51 AM, Pascal Bach <pascal.bach@siemens.com> wrote:
>>>> This was probably working before pyro and the recipes specific sysroots.
>>>>
>>>> Signed-off-by: Pascal Bach <pascal.bach@siemens.com>
>>>> ---
>>>>   recipes-mono/mono/mono-4.xx.inc | 2 +-
>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/recipes-mono/mono/mono-4.xx.inc b/recipes-mono/mono/mono-4.xx.inc
>>>> index e06fac7..cf9170b 100644
>>>> --- a/recipes-mono/mono/mono-4.xx.inc
>>>> +++ b/recipes-mono/mono/mono-4.xx.inc
>>>> @@ -4,7 +4,7 @@ HOMEPAGE = "http://mono-project.com"
>>>>   BUGTRACKER = "http://bugzilla.xamarin.com/"
>>>>   SECTION = "devel"
>>>>
>>>> -DEPENDS = "zlib"
>>>> +DEPENDS = "zlib cmake-native"
>>>>
>>> perhaps this should inehrit cmake if its using cmake
>>> or explain how cmake is used where abstactions
>>> are not relevant
>> Using the cmake bbclass doesn't work as it is not really a CMake project.
>>
>> 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)

Cheers,

Alex

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Pascal Bach]

 

>>>
>>> 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

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Alex Lennon]

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

Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake

[Khem Raj]

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
>