Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-23 Thread Alan Modra
On Fri, Aug 23, 2013 at 10:39:42AM +0200, Peter Rosin wrote: > On 2013-08-22 17:48, Alan Modra wrote: > > On Thu, Aug 22, 2013 at 09:34:10PM +0700, Gary V. Vaughan wrote: > >>> How can it be correct to say "-m elf32lppclinux" (32-bit) when $host is > >>> explicitly 64-bit? That seems like utter gar

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-23 Thread Peter Rosin
On 2013-08-22 17:48, Alan Modra wrote: > On Thu, Aug 22, 2013 at 09:34:10PM +0700, Gary V. Vaughan wrote: >>> How can it be correct to say "-m elf32lppclinux" (32-bit) when $host is >>> explicitly 64-bit? That seems like utter garbage to me. What am I >>> missing this time? > > As far as I underst

Re: [patch] Comments for linker-option section, was Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
Hi Brooks, On Aug 23, 2013, at 5:52 AM, Brooks Moses wrote: > Based on Alan's explanation and what I could tell from looking at the code, > I've tried to expand the comments to explain what's going on, as per the > attached patch. Does this look like a useful improvement to the situation? Per

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Alan Modra
On Fri, Aug 23, 2013 at 11:10:09AM +0700, Gary V. Vaughan wrote: > Thanks for the explanation, I finally do get it. Phew :) Oh good, because I'm not sure I really understood it despite writing an explanation. :) > I believe I already fixed it with the most recent amendment committed > under you

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
Hi Alan, Thanks for the fast feedback. On Aug 22, 2013, at 10:48 PM, Alan Modra wrote: > On Thu, Aug 22, 2013 at 09:34:10PM +0700, Gary V. Vaughan wrote: >>> How can it be correct to say "-m elf32lppclinux" (32-bit) when $host is >>> explicitly 64-bit? That seems like utter garbage to me. What a

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Alan Modra
On Thu, Aug 22, 2013 at 09:34:10PM +0700, Gary V. Vaughan wrote: > > How can it be correct to say "-m elf32lppclinux" (32-bit) when $host is > > explicitly 64-bit? That seems like utter garbage to me. What am I > > missing this time? As far as I understand, this piece of libtool is supplying ld op

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
Hi Peter, On Aug 22, 2013, at 8:58 PM, Peter Rosin wrote: > On 2013-08-22 10:20, Gary V. Vaughan wrote: >> On Aug 22, 2013, at 2:54 PM, Peter Rosin wrote: >>> On 2013-08-22 09:40, Gary V. Vaughan wrote: > Can we please get this simple patch pushed? Done. >>> >>> To me, it appears

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Peter Rosin
On 2013-08-22 10:20, Gary V. Vaughan wrote: > Hi Peter, > > On Aug 22, 2013, at 2:54 PM, Peter Rosin wrote: > >> On 2013-08-22 09:40, Gary V. Vaughan wrote: Can we please get this simple patch pushed? >>> >>> Done. >> >> To me, it appears as if what you actually pushed was not what was post

Re: powerpc*le-linux support

2013-08-22 Thread Peter Rosin
On 2013-08-22 15:25, Alan Modra wrote: > On Thu, Aug 22, 2013 at 01:16:04PM +0200, Peter Rosin wrote: >> I guess I'm just thoroughly confused, but in my world there ought to >> be four variations of $host; 64- or 32-bit, and big or little endian. >> >> This patch seems to only handle builds going f

Re: powerpc*le-linux support

2013-08-22 Thread Alan Modra
On Thu, Aug 22, 2013 at 01:16:04PM +0200, Peter Rosin wrote: > I guess I'm just thoroughly confused, but in my world there ought to > be four variations of $host; 64- or 32-bit, and big or little endian. > > This patch seems to only handle builds going from 64-bit to 32-bit > ($host powerpc64-* an

Re: powerpc*le-linux support

2013-08-22 Thread Peter Rosin
On 2013-08-22 10:19, Gary V. Vaughan wrote: > On Aug 22, 2013, at 2:59 PM, Peter Rosin wrote: >> On 2013-06-06 07:18, Alan Modra wrote: *snip* >>> diff --git a/m4/libtool.m4 b/m4/libtool.m4 >>> index d7013c5..501246d 100644 >>> --- a/m4/libtool.m4 >>> +++ b/m4/libtool.m4 >>> @@ -1307,7 +1307,7 @

Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
Hi Peter, On Aug 22, 2013, at 3:19 PM, "Gary V. Vaughan" wrote: >> All other inner cases match one of the outer or-ed expressions (i.e. >> from the first hunk) quite closely, but the outer match is still >> powerpc*-*linux* while the inner match has dropped the trailing >> -*linux* part. I would

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
Hi Peter, On Aug 22, 2013, at 2:54 PM, Peter Rosin wrote: > On 2013-08-22 09:40, Gary V. Vaughan wrote: >>> Can we please get this simple patch pushed? >> >> Done. > > To me, it appears as if what you actually pushed was not what was posted? I am an idiot. Thanks for the heads up, fixed in t

Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
On Aug 22, 2013, at 2:59 PM, Peter Rosin wrote: > Hi! > > Ok, Gary pushed something while I wrote this, but I'm sending it > anyway since what he pushed didn't look quite right to me… Argh. I pasted the patch from my browser to a terminal to save hunting my archives for the original email, and

Re: powerpc*le-linux support

2013-08-22 Thread Peter Rosin
Hi! Ok, Gary pushed something while I wrote this, but I'm sending it anyway since what he pushed didn't look quite right to me... On 2013-06-06 07:18, Alan Modra wrote: > On Thu, Jun 06, 2013 at 11:31:34AM +0930, Alan Modra wrote: >> This adds support for little-endian powerpc linux, and tidies t

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Peter Rosin
On 2013-08-22 09:40, Gary V. Vaughan wrote: >> Can we please get this simple patch pushed? > > Done. To me, it appears as if what you actually pushed was not what was posted? Cheers, Peter

Re: Two-month patch ping: Re: powerpc*le-linux support

2013-08-22 Thread Gary V. Vaughan
Hi Brooks, Alan, On Aug 22, 2013, at 12:59 AM, Brooks Moses wrote: > Ping? > > (And adding libtool@, in hopes of getting a little more attention, as pings > on libtool-patches@ seem to be going nowhere.) > > On 06/05/2013 07:01 PM, Alan Modra wrote: >> This adds support for little-endian powe

Re: powerpc*le-linux support

2013-08-08 Thread Alan Modra
On Thu, Jul 18, 2013 at 08:31:03AM +0930, Alan Modra wrote: > Ping? > http://lists.gnu.org/archive/html/libtool-patches/2013-06/msg1.html > > On Thu, Jun 06, 2013 at 02:48:22PM +0930, Alan Modra wrote: > > * m4/libtool.m4 (ld -m flags): Remove non-canonical ppc host match. > > Support

Re: powerpc*le-linux support

2013-07-22 Thread Pavel Raiskup
> Haven't found enough time to do the review .. but I should have to try to find at least a second. I see patch is relevant to libtool, sorry for my previous pointless comment.

Re: powerpc*le-linux support

2013-07-22 Thread Pavel Raiskup
> [...] config.sub won't return ppc*-*linux* so there isn't much point in > matching that. [...] Haven't found enough time to do the review but issues with config.{guess,sub} should be discussed on different list: config-patc...@gnu.org Pavel

Re: powerpc*le-linux support

2013-07-17 Thread Alan Modra
Ping? http://lists.gnu.org/archive/html/libtool-patches/2013-06/msg1.html On Thu, Jun 06, 2013 at 02:48:22PM +0930, Alan Modra wrote: > * m4/libtool.m4 (ld -m flags): Remove non-canonical ppc host match. > Support little-endian powerpc linux host. -- Alan Modra Australia Developm

powerpc*le-linux support

2013-06-05 Thread Alan Modra
This adds support for little-endian powerpc linux, and tidies the existing host match for powerpc. config.sub won't return ppc*-*linux* so there isn't much point in matching that. * m4/libtool.m4 (ld -m flags): Remove non-canonical ppc host match. Support little-endian powerpc lin

Re: powerpc*le-linux support

2013-06-05 Thread Alan Modra
On Thu, Jun 06, 2013 at 11:31:34AM +0930, Alan Modra wrote: > This adds support for little-endian powerpc linux, and tidies the > existing host match for powerpc. config.sub won't return ppc*-*linux* > so there isn't much point in matching that. > - ppc*-*linux*|powerpc*-*linux*) > +