Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Mark Hatle <mark.hatle@windriver.com>
To: Khem Raj <raj.khem@gmail.com>, Martin Jansa <martin.jansa@gmail.com>
Cc: Otavio Salvador <otavio.salvador@ossystems.com.br>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/6] gmp: Bring back version 4.2.1 (LGPL 2.1+)
Date: Tue, 1 Sep 2015 09:55:28 -0500	[thread overview]
Message-ID: <55E5BC60.4010302@windriver.com> (raw)
In-Reply-To: <BC8B1BD5-224C-480C-BFC9-F2BA6B0254E9@gmail.com>

On 8/31/15 10:27 AM, Khem Raj wrote:
> 
>> On Aug 31, 2015, at 8:21 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
>>
>> On Mon, Aug 31, 2015 at 10:22:31AM -0300, Otavio Salvador wrote:
>>> On Sat, Aug 29, 2015 at 7:18 AM, Richard Purdie
>>> <richard.purdie@linuxfoundation.org> wrote:
>>>> On Sat, 2015-08-29 at 10:43 +0200, Martin Jansa wrote:
>>>>> Did you enable thumb in that qemuarm build?
>>>>>
>>>>>
>>>>> It's disabled by default and multiple components are broken when you
>>>>> enable it, see:
>>>>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=7717
>>>>
>>>> I'm going to draw a line here. People are asking that the old version of
>>>> gmp be brought back due license issues, fine. It doesn't mean that all
>>>> the bugs in said old release should be fixed.
>>>>
>>>> If thumb on arm with old gmp is an issue for people, I'd suggest that
>>>> those people figure out which patches are needed and send the patches.
>>>> Asking Jussi to deal with this isn't right/fair though.
>>>
>>> I disagree. If it will be merged it ought to support what we are
>>> testing nowadays otherwise the backlog will be kept growing forever...
>>
>> FWIW: I don't mind getting it merged as is.
>>
>> It improves situation for non-GPLv3 builds and if there is issue with
>> thumb, then it was there before as well, so returning it doesn't make
>> things worse.
>>
>> My previous reply was just to show that building default qemuarm doesn't
>> show anything, because thumb isn't enabled and gmp even isn't one of the
>> recipes which are failing with thumb-1, maybe Khem meant some issues in
>> runtime I don't know about.
> 
> It has issues working with thumb2 as well. Another problem it creates is building on-device toolchain
> since gcc also depends on gmp and when you pin to older version of GMP, it starts to fail
> Thats why I suggested to use mini-gmp ( same but reduced gmp ) can clear of this situation since it
> will be a mini-gmp recipe and won’t interfere with toolchain.
> 

Do you have any sense of the problems when gcc uses an older version of GMP?  If
a newer version is required for gcc -- then can gcc itself check that it's a
proper minimum version available?  (At this point I would assume if someone is
worried about GPLv3 issues, that they would NOT be putting the compiler and
related tools on the target.)  I know we can specify -runtime- minimum versions
easily enough using the > directive.. but I don't think there is an easy way to
specify minimum build version, other then verify we have the correct version
available to us though a file check of some kind.

--Mark


  reply	other threads:[~2015-09-01 14:55 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-27 13:05 [PATCH 0/6] Bring back LGPLv2 gmp, upgrade gnutls stack Jussi Kukkonen
2015-08-27 13:05 ` [PATCH 1/6] nettle: Fix binary license to LGPLv2.1+ Jussi Kukkonen
2015-08-27 13:05 ` [PATCH 2/6] gmp: Bring back version 4.2.1 (LGPL 2.1+) Jussi Kukkonen
2015-08-28  6:03   ` Khem Raj
2015-08-28  7:44     ` Jussi Kukkonen
2015-08-29  6:17       ` Jussi Kukkonen
2015-08-29  8:43         ` Martin Jansa
2015-08-29 10:18           ` Richard Purdie
2015-08-31 13:22             ` Otavio Salvador
2015-08-31 13:53               ` Richard Purdie
2015-08-31 14:01                 ` Otavio Salvador
2015-08-31 14:10                   ` Alexander Kanavin
2015-08-31 14:38                 ` Khem Raj
2015-09-01 11:21                   ` Jussi Kukkonen
2015-09-01 14:45                     ` Khem Raj
2015-08-31 15:21               ` Martin Jansa
2015-08-31 15:27                 ` Khem Raj
2015-09-01 14:55                   ` Mark Hatle [this message]
2015-09-01 17:10                     ` Khem Raj
2015-08-27 13:06 ` [PATCH 3/6] nettle: Add (LGPLv3) version 3.1.1 Jussi Kukkonen
2015-08-27 13:06 ` [PATCH 4/6] nettle: Add ptests Jussi Kukkonen
2015-08-28 12:16   ` Jussi Kukkonen
2015-08-27 13:06 ` [PATCH 5/6] gnutls: Upgrade 3.3.14 -> 3.3.17.1 Jussi Kukkonen
2015-08-27 13:06 ` [PATCH 6/6] mirrors.bbclass: Update gnutls mirrors Jussi Kukkonen
2015-08-27 17:13   ` Christopher Larson
2015-08-31 11:27     ` Richard Purdie

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=55E5BC60.4010302@windriver.com \
    --to=mark.hatle@windriver.com \
    --cc=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio.salvador@ossystems.com.br \
    --cc=raj.khem@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox