All of lore.kernel.org
 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 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.