From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hetzner.pbcl.net (mail.pbcl.net [88.198.119.4]) by mail.openembedded.org (Postfix) with ESMTP id 6A8816010B for ; Tue, 17 Nov 2015 17:09:42 +0000 (UTC) Received: from [2a00:14f0:e04c:1:e23f:49ff:fe47:b7fd] (helo=phil-desktop) by hetzner.pbcl.net with esmtpsa (TLS1.2:RSA_AES_128_CBC_SHA1:128) (Exim 4.80) (envelope-from ) id 1ZyjlG-0006WJ-3i; Tue, 17 Nov 2015 18:09:42 +0100 Message-ID: <1447780178.1472.29.camel@pbcl.net> From: Phil Blundell To: "Burton, Ross" Date: Tue, 17 Nov 2015 17:09:38 +0000 In-Reply-To: References: <1447739760-14395-1-git-send-email-armccurdy@gmail.com> <1447773377.1472.28.camel@pbcl.net> X-Mailer: Evolution 3.16.5-1 Mime-Version: 1.0 Cc: OE-core Subject: Re: [PATCH] directfb.inc: force bfd linker for armv7a X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Nov 2015 17:09:47 -0000 Content-Type: multipart/alternative; boundary="=-TI9qYjqriTtY4UBU9krn" --=-TI9qYjqriTtY4UBU9krn Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Tue, 2015-11-17 at 16:44 +0000, Burton, Ross wrote: > > On 17 November 2015 at 15:16, Phil Blundell wrote: > > Does the resulting directfb-csource binary actually work with that > > change? My recollection is that ld.bfd will just silently permit > > relocs that gold (correctly) refuses to allow. > > > Personally I don't use DirectFB (and am constantly surprised it's > still a thing) so at this point I'm not going to merge either of > these patches until someone can confirm that their fix *works on > hardware* instead of just fixing the build. I think that's a reasonable position. In any case, the underlying issue is a compiler bug and ought really to be fixed there rather than patching directfb or any other package that just happens to fall over it. p. --=-TI9qYjqriTtY4UBU9krn Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 7bit
On Tue, 2015-11-17 at 16:44 +0000, Burton, Ross wrote:

On 17 November 2015 at 15:16, Phil Blundell <pb@pbcl.net> wrote:
Does the resulting directfb-csource binary actually work with that
change?  My recollection is that ld.bfd will just silently permit
relocs that gold (correctly) refuses to allow.


Personally I don't use DirectFB (and am constantly surprised it's still a thing) so at this point I'm not going to merge either of these patches until someone can confirm that their fix *works on hardware* instead of just fixing the build.

I think that's a reasonable position.  In any case, the underlying issue is a compiler bug and ought really to be fixed there rather than patching directfb or any other package that just happens to fall over it.

p.

--=-TI9qYjqriTtY4UBU9krn--