diff mbox

fix PR65726

Message ID 5657544E.1080204@fgznet.ch
State New
Headers show

Commit Message

Andreas Tobler Nov. 26, 2015, 6:49 p.m. UTC
Hi all,

the attached patch fixes the build issue from this ticket if bootstrap 
is disabled.

Tested on x86_64-*-linux* and on x86_64-*-freebsd* with gcc and clang.

Ok for trunk?

And 5.3?

Thanks,
Andreas

2015-11-26  Andreas Tobler  <andreast@gcc.gnu.org>

	PR libffi/65726
	* Makefile.def (lang_env_dependencies): Make libffi depend
	on cxx.
	* Makefile.in: Regenerate.

Comments

Jeff Law Nov. 30, 2015, 10:30 p.m. UTC | #1
On 11/26/2015 11:49 AM, Andreas Tobler wrote:
> Hi all,
>
> the attached patch fixes the build issue from this ticket if bootstrap
> is disabled.
>
> Tested on x86_64-*-linux* and on x86_64-*-freebsd* with gcc and clang.
>
> Ok for trunk?
>
> And 5.3?
>
> Thanks,
> Andreas
>
> 2015-11-26  Andreas Tobler  <andreast@gcc.gnu.org>
>
>      PR libffi/65726
>      * Makefile.def (lang_env_dependencies): Make libffi depend
>      on cxx.
>      * Makefile.in: Regenerate.
>
OK.
jeff
Andreas Tobler Dec. 1, 2015, 9:50 p.m. UTC | #2
On 30.11.15 23:30, Jeff Law wrote:
> On 11/26/2015 11:49 AM, Andreas Tobler wrote:
>> Hi all,
>>
>> the attached patch fixes the build issue from this ticket if bootstrap
>> is disabled.
>>
>> Tested on x86_64-*-linux* and on x86_64-*-freebsd* with gcc and clang.
>>
>> Ok for trunk?
>>
>> And 5.3?
>>
>> Thanks,
>> Andreas
>>
>> 2015-11-26  Andreas Tobler  <andreast@gcc.gnu.org>
>>
>>       PR libffi/65726
>>       * Makefile.def (lang_env_dependencies): Make libffi depend
>>       on cxx.
>>       * Makefile.in: Regenerate.
>>
> OK.


Thanks!

Committed to trunk. I'll wait till gcc5 opens again and then I commit to 
gcc5 and gcc49, ok?

Andreas
Jeff Law Dec. 1, 2015, 9:55 p.m. UTC | #3
On 12/01/2015 02:50 PM, Andreas Tobler wrote:
> On 30.11.15 23:30, Jeff Law wrote:
>> On 11/26/2015 11:49 AM, Andreas Tobler wrote:
>>> Hi all,
>>>
>>> the attached patch fixes the build issue from this ticket if bootstrap
>>> is disabled.
>>>
>>> Tested on x86_64-*-linux* and on x86_64-*-freebsd* with gcc and clang.
>>>
>>> Ok for trunk?
>>>
>>> And 5.3?
>>>
>>> Thanks,
>>> Andreas
>>>
>>> 2015-11-26  Andreas Tobler  <andreast@gcc.gnu.org>
>>>
>>>       PR libffi/65726
>>>       * Makefile.def (lang_env_dependencies): Make libffi depend
>>>       on cxx.
>>>       * Makefile.in: Regenerate.
>>>
>> OK.
>
>
> Thanks!
>
> Committed to trunk. I'll wait till gcc5 opens again and then I commit to
> gcc5 and gcc49, ok?
Seems reasonable.
jeff
Andreas Tobler Dec. 4, 2015, 8:51 p.m. UTC | #4
On 01.12.15 22:55, Jeff Law wrote:
> On 12/01/2015 02:50 PM, Andreas Tobler wrote:
>> On 30.11.15 23:30, Jeff Law wrote:
>>> On 11/26/2015 11:49 AM, Andreas Tobler wrote:
>>>> Hi all,
>>>>
>>>> the attached patch fixes the build issue from this ticket if bootstrap
>>>> is disabled.
>>>>
>>>> Tested on x86_64-*-linux* and on x86_64-*-freebsd* with gcc and clang.
>>>>
>>>> Ok for trunk?
>>>>
>>>> And 5.3?
>>>>
>>>> Thanks,
>>>> Andreas
>>>>
>>>> 2015-11-26  Andreas Tobler  <andreast@gcc.gnu.org>
>>>>
>>>>        PR libffi/65726
>>>>        * Makefile.def (lang_env_dependencies): Make libffi depend
>>>>        on cxx.
>>>>        * Makefile.in: Regenerate.
>>>>
>>> OK.
>>
>>
>> Thanks!
>>
>> Committed to trunk. I'll wait till gcc5 opens again and then I commit to
>> gcc5 and gcc49, ok?
> Seems reasonable.


Thank you!

PR closed.

Andreas
diff mbox

Patch

Index: Makefile.def
===================================================================
--- Makefile.def	(revision 230942)
+++ Makefile.def	(working copy)
@@ -534,6 +534,7 @@ 
 // on libgcc and newlib/libgloss.
 lang_env_dependencies = { module=libjava; cxx=true; };
 lang_env_dependencies = { module=libitm; cxx=true; };
+lang_env_dependencies = { module=libffi; cxx=true; };
 lang_env_dependencies = { module=libcilkrts; cxx=true; };
 lang_env_dependencies = { module=liboffloadmic; cxx=true; };
 lang_env_dependencies = { module=newlib; no_c=true; };
Index: Makefile.in
===================================================================
--- Makefile.in	(revision 230942)
+++ Makefile.in	(working copy)
@@ -51034,6 +51034,7 @@ 
 
 
 configure-target-libffi: maybe-all-target-newlib maybe-all-target-libgloss
+configure-target-libffi: maybe-all-target-libstdc++-v3
 
 configure-target-libjava: maybe-all-target-newlib maybe-all-target-libgloss
 configure-target-libjava: maybe-all-target-libstdc++-v3