diff mbox series

configure: Support building D front-end on *-*-darwin*

Message ID 20201128184400.85153-1-ibuclaw@gdcproject.org
State New
Headers show
Series configure: Support building D front-end on *-*-darwin* | expand

Commit Message

Iain Buclaw Nov. 28, 2020, 6:44 p.m. UTC
Hi,

The bootstrap has been succeeding for some time now, there's no need to
set it as an unsupported language.

OK for mainline?

Regards
Iain.

---
ChangeLog:

	PR d/87788
	* configure.ac: Don't disable D for *-*-darwin*.
	* configure: Regenerate.
---
 configure    | 3 ---
 configure.ac | 3 ---
 2 files changed, 6 deletions(-)

Comments

Iain Sandoe Nov. 29, 2020, 9:35 a.m. UTC | #1
Hi Iain

Iain Buclaw <ibuclaw@gdcproject.org> wrote:

> The bootstrap has been succeeding for some time now, there's no need to
> set it as an unsupported language.
>
> OK for mainline?

At present, this breaks bootstrap on 32 bit Darwin hosts.

Once that’s resolved, then it seems a reasonable to make sure that the D
FE is built and tested (even though there is still work to do on the library
support).

So, OK once all the currently tested Darwin hosts bootstrap with D enabled.

thanks!
Iai

>
> Regards
> Iain.
>
> ---
> ChangeLog:
>
> 	PR d/87788
> 	* configure.ac: Don't disable D for *-*-darwin*.
> 	* configure: Regenerate.
> ---
> configure    | 3 ---
> configure.ac | 3 ---
> 2 files changed, 6 deletions(-)
>
> diff --git a/configure b/configure
> index b41a258c80e..d07988312b1 100755
> --- a/configure
> +++ b/configure
> @@ -3444,9 +3444,6 @@ case ,${enable_languages}, in
>     ;;
>   *)
>     case "${target}" in
> -      *-*-darwin*)
> -	unsupported_languages="$unsupported_languages d"
> -        ;;
>       bpf-*-*)
> 	unsupported_languages="$unsupported_languages d"
> 	;;
> diff --git a/configure.ac b/configure.ac
> index 944f30cfea8..12f85cc25fa 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -724,9 +724,6 @@ case ,${enable_languages}, in
>     ;;
>   *)
>     case "${target}" in
> -      *-*-darwin*)
> -	unsupported_languages="$unsupported_languages d"
> -        ;;
>       bpf-*-*)
> 	unsupported_languages="$unsupported_languages d"
> 	;;
> -- 
> 2.27.0
Iain Buclaw Nov. 29, 2020, 1:33 p.m. UTC | #2
Excerpts from Iain Sandoe's message of November 29, 2020 10:35 am:
> Hi Iain
> 
> Iain Buclaw <ibuclaw@gdcproject.org> wrote:
> 
>> The bootstrap has been succeeding for some time now, there's no need to
>> set it as an unsupported language.
>>
>> OK for mainline?
> 
> At present, this breaks bootstrap on 32 bit Darwin hosts.
> 
> Once that’s resolved, then it seems a reasonable to make sure that the D
> FE is built and tested (even though there is still work to do on the library
> support).
> 
> So, OK once all the currently tested Darwin hosts bootstrap with D enabled.
> 

As of r11-5516 the bootstrap should be fixed now.

Iain.
Iain Buclaw Nov. 29, 2020, 9:13 p.m. UTC | #3
Excerpts from Iain Sandoe's message of November 29, 2020 10:35 am:
> Hi Iain
> 
> Iain Buclaw <ibuclaw@gdcproject.org> wrote:
> 
>> The bootstrap has been succeeding for some time now, there's no need to
>> set it as an unsupported language.
>>
>> OK for mainline?
> 
> At present, this breaks bootstrap on 32 bit Darwin hosts.
> 
> Once that’s resolved, then it seems a reasonable to make sure that the D
> FE is built and tested (even though there is still work to do on the library
> support).
> 
> So, OK once all the currently tested Darwin hosts bootstrap with D enabled.
> 

As confirmed off list, powerpc and i686 darwin9 bootstrapped OK, so I've
committed it as r11-5521.

Iain.
diff mbox series

Patch

diff --git a/configure b/configure
index b41a258c80e..d07988312b1 100755
--- a/configure
+++ b/configure
@@ -3444,9 +3444,6 @@  case ,${enable_languages}, in
     ;;
   *)
     case "${target}" in
-      *-*-darwin*)
-	unsupported_languages="$unsupported_languages d"
-        ;;
       bpf-*-*)
 	unsupported_languages="$unsupported_languages d"
 	;;
diff --git a/configure.ac b/configure.ac
index 944f30cfea8..12f85cc25fa 100644
--- a/configure.ac
+++ b/configure.ac
@@ -724,9 +724,6 @@  case ,${enable_languages}, in
     ;;
   *)
     case "${target}" in
-      *-*-darwin*)
-	unsupported_languages="$unsupported_languages d"
-        ;;
       bpf-*-*)
 	unsupported_languages="$unsupported_languages d"
 	;;