This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] Fix binutils build on sparc.

From: Alan Modra <amodra at bigpond dot net dot au>
To: Kazu Hirata <kazu at codesourcery dot com>
Cc: binutils at sourceware dot org, joseph at codesourcery dot com
Date: Wed, 15 Apr 2009 12:36:41 +0930
Subject: Re: [patch] Fix binutils build on sparc.
References: <20090414191752.CD30D6F62057@daisy.codesourcery.com> <20090415002722.GG27796@bubble.grove.modra.org> <49E5497E.5050100@codesourcery.com>

On Tue, Apr 14, 2009 at 10:42:06PM -0400, Kazu Hirata wrote:
> Hi Alan,
>
> Thank you for a quick review.
>
>>> ld/
>>> 2009-04-14  Kazu Hirata  <kazu@codesourcery.com>
>>>
>>> 	* ldlang.c: Include config.h.
>>
>> No, just move #include <limits.h> later in the file.  sysdep.h should
>> always be first.
>
> OK.  Could I just drop #include <limits.h> in ldlang.c? sysdep.h seems to 
> do it all -- include "config.h" first and then <limits.h> if available.

Yes please.

-- 
Alan Modra
Australia Development Lab, IBM

References:
- [patch] Fix binutils build on sparc.
  - From: Kazu Hirata
- Re: [patch] Fix binutils build on sparc.
  - From: Alan Modra
- Re: [patch] Fix binutils build on sparc.
  - From: Kazu Hirata

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]