* [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
@ 2017-10-27 10:51 Pascal Bach
2017-10-27 21:41 ` Khem Raj
0 siblings, 1 reply; 8+ messages in thread
From: Pascal Bach @ 2017-10-27 10:51 UTC (permalink / raw)
To: yocto, ajlennon
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
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
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
0 siblings, 1 reply; 8+ messages in thread
From: Khem Raj @ 2017-10-27 21:41 UTC (permalink / raw)
To: Pascal Bach; +Cc: yocto
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
2017-10-27 21:41 ` Khem Raj
@ 2017-11-02 15:46 ` Pascal Bach
2017-11-02 15:59 ` Khem Raj
0 siblings, 1 reply; 8+ messages in thread
From: Pascal Bach @ 2017-11-02 15:46 UTC (permalink / raw)
To: Khem Raj; +Cc: yocto
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
2017-11-02 15:46 ` Pascal Bach
@ 2017-11-02 15:59 ` Khem Raj
2017-11-03 11:48 ` Alex Lennon
0 siblings, 1 reply; 8+ messages in thread
From: Khem Raj @ 2017-11-02 15:59 UTC (permalink / raw)
To: Pascal Bach; +Cc: yocto
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
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
2017-11-02 15:59 ` Khem Raj
@ 2017-11-03 11:48 ` Alex Lennon
2017-11-07 13:26 ` Pascal Bach
0 siblings, 1 reply; 8+ messages in thread
From: Alex Lennon @ 2017-11-03 11:48 UTC (permalink / raw)
To: Khem Raj, Pascal Bach; +Cc: yocto
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
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
0 siblings, 2 replies; 8+ messages in thread
From: Pascal Bach @ 2017-11-07 13:26 UTC (permalink / raw)
To: Alex Lennon, Khem Raj; +Cc: yocto
>>>
>>> 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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
2017-11-07 13:26 ` Pascal Bach
@ 2017-11-07 13:57 ` Alex Lennon
2017-11-07 16:36 ` Khem Raj
1 sibling, 0 replies; 8+ messages in thread
From: Alex Lennon @ 2017-11-07 13:57 UTC (permalink / raw)
To: Pascal Bach, Khem Raj; +Cc: yocto
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-mono][PATCH] mono-4.xx: compiling mono 4 with btls requires cmake
2017-11-07 13:26 ` Pascal Bach
2017-11-07 13:57 ` Alex Lennon
@ 2017-11-07 16:36 ` Khem Raj
1 sibling, 0 replies; 8+ messages in thread
From: Khem Raj @ 2017-11-07 16:36 UTC (permalink / raw)
To: Pascal Bach; +Cc: yocto
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
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-11-07 16:37 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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 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.