diff mbox

fix bootstrap in libgcc for PowerPC FreeBSD

Message ID 4E3C605E.3070105@fgznet.ch
State New
Headers show

Commit Message

Andreas Tobler Aug. 5, 2011, 9:27 p.m. UTC 
Hi,

the patch below survives bootstrap stage-1 on PowerPC FreeBSD successfully.
W/o patch we try to fliter-out ibm-ldouble.c. So from my understanding 
the pattern ibm-ldouble.c does never match since there is a path 
prepended to the file name.
With the patch we look for everything which contains ibm-ldouble.c...

Is this ok for head?

Thanks,
Andreas

2011-08-05  Andreas Tobler  <andreast@fgznet.ch>

	* config/rs6000/t-freebsd: Add a wildcard to the filter to find
	the file with the path.

Comments

Rainer Orth Aug. 9, 2011, 11:59 a.m. UTC | #1 
Hi Andreas,

> the patch below survives bootstrap stage-1 on PowerPC FreeBSD successfully.
> W/o patch we try to fliter-out ibm-ldouble.c. So from my understanding the
> pattern ibm-ldouble.c does never match since there is a path prepended to
> the file name.
> With the patch we look for everything which contains ibm-ldouble.c...
>
> Is this ok for head?

while I cannot approve it, I'm pretty sure this counts as obvious.

Sorry for the breakage.

	Rainer


> 2011-08-05  Andreas Tobler  <andreast@fgznet.ch>
>
> 	* config/rs6000/t-freebsd: Add a wildcard to the filter to find
> 	the file with the path.

I'd write it as

	* config/rs6000/t-freebsd (LIB2ADD): Add wildcard.

> Index: config/rs6000/t-freebsd
> ===================================================================
> --- config/rs6000/t-freebsd	(revision 177472)
> +++ config/rs6000/t-freebsd	(working copy)
> @@ -19,4 +19,4 @@
>  # <http://www.gnu.org/licenses/>.
>
>  # We do not want to build ibm-ldouble.c.
> -LIB2ADD := $(filter-out ibm-ldouble.c, $(LIB2ADD))
> +LIB2ADD := $(filter-out %ibm-ldouble.c, $(LIB2ADD))
Andreas Tobler Oct. 25, 2011, 5:14 a.m. UTC | #2 
On 09.08.11 13:59, Rainer Orth wrote:
> Hi Andreas,
>
>> the patch below survives bootstrap stage-1 on PowerPC FreeBSD successfully.
>> W/o patch we try to fliter-out ibm-ldouble.c. So from my understanding the
>> pattern ibm-ldouble.c does never match since there is a path prepended to
>> the file name.
>> With the patch we look for everything which contains ibm-ldouble.c...
>>
>> Is this ok for head?
>
> while I cannot approve it, I'm pretty sure this counts as obvious.

I committed this one.

Thanks,
Andreas
>
> Sorry for the breakage.
>
> 	Rainer
>
>
>> 2011-08-05  Andreas Tobler<andreast@fgznet.ch>
>>
>> 	* config/rs6000/t-freebsd: Add a wildcard to the filter to find
>> 	the file with the path.
>
> I'd write it as
>
> 	* config/rs6000/t-freebsd (LIB2ADD): Add wildcard.
>
>> Index: config/rs6000/t-freebsd
>> ===================================================================
>> --- config/rs6000/t-freebsd	(revision 177472)
>> +++ config/rs6000/t-freebsd	(working copy)
>> @@ -19,4 +19,4 @@
>>   #<http://www.gnu.org/licenses/>.
>>
>>   # We do not want to build ibm-ldouble.c.
>> -LIB2ADD := $(filter-out ibm-ldouble.c, $(LIB2ADD))
>> +LIB2ADD := $(filter-out %ibm-ldouble.c, $(LIB2ADD))
>
diff mbox

Patch

Index: config/rs6000/t-freebsd
===================================================================
--- config/rs6000/t-freebsd	(revision 177472)
+++ config/rs6000/t-freebsd	(working copy)
@@ -19,4 +19,4 @@ 
  # <http://www.gnu.org/licenses/>.

  # We do not want to build ibm-ldouble.c.
-LIB2ADD := $(filter-out ibm-ldouble.c, $(LIB2ADD))
+LIB2ADD := $(filter-out %ibm-ldouble.c, $(LIB2ADD))