public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <bonzini@gnu.org>
To: Jakub Jelinek <jakub@redhat.com>,
	 Dominique Dhumieres <dominiq@lps.ens.fr>
Cc: gcc-patches@gcc.gnu.org, gcc@gcc.gnu.org, fxcoudert@gmail.com,
	 pmuldoon@redhat.com, iain@codesourcery.com
Subject: Re: [patch, build] Restore bootstrap in building libcc1 on darwin
Date: Sat, 13 Dec 2014 10:52:00 -0000	[thread overview]
Message-ID: <548C1A51.9020301@gnu.org> (raw)
In-Reply-To: <20141205224751.GS1923@tucnak.redhat.com>



On 05/12/2014 23:47, Jakub Jelinek wrote:
> On Fri, Dec 05, 2014 at 11:34:28PM +0100, Dominique Dhumieres wrote:
>>> As I've tried to explain, that is IMHO wrong though.
>>> If what you are after is the -B stuff too, then perhaps:
>>> ...
>>
>> Sorry but it does not work:
> 
> Sorry, make that (just removed 4x ' in each file):
> 
> 2014-12-05  Jakub Jelinek  <jakub@redhat.com>
> 
> 	PR bootstrap/64023
> 	* Makefile.tpl (EXTRA_TARGET_FLAGS): Set STAGE1_LDFLAGS
> 	to POSTSTAGE1_LDFLAGS and STAGE1_LIBS to POSTSTAGE1_LIBS.
> 	Add -B to libstdc++-v3/src/.libs and libstdc++-v3/libsupc++/.libs
> 	to CXX.
> 	* Makefile.in: Regenerated.
> 
> --- Makefile.tpl.jj	2014-11-12 09:31:59.000000000 +0100
> +++ Makefile.tpl	2014-12-05 21:12:21.486031062 +0100
> @@ -641,7 +641,9 @@ EXTRA_TARGET_FLAGS = \
>  	'AS=$(COMPILER_AS_FOR_TARGET)' \
>  	'CC=$$(CC_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CFLAGS=$$(CFLAGS_FOR_TARGET)' \
> -	'CXX=$$(CXX_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> +	'CXX=$$(CXX_FOR_TARGET) -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
> +	 -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs \
> +	 $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CXXFLAGS=$$(CXXFLAGS_FOR_TARGET)' \
>  	'DLLTOOL=$$(DLLTOOL_FOR_TARGET)' \
>  	'GCJ=$$(GCJ_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> @@ -659,6 +661,8 @@ EXTRA_TARGET_FLAGS = \
>  	'WINDRES=$$(WINDRES_FOR_TARGET)' \
>  	'WINDMC=$$(WINDMC_FOR_TARGET)' \
>  	'XGCC_FLAGS_FOR_TARGET=$(XGCC_FLAGS_FOR_TARGET)' \
> +	'STAGE1_LDFLAGS=$$(POSTSTAGE1_LDFLAGS)' \
> +	'STAGE1_LIBS=$$(POSTSTAGE1_LIBS)' \
>  	"TFLAGS=$$TFLAGS"
>  
>  TARGET_FLAGS_TO_PASS = $(BASE_FLAGS_TO_PASS) $(EXTRA_TARGET_FLAGS)
> --- Makefile.in.jj	2014-11-28 14:40:52.000000000 +0100
> +++ Makefile.in	2014-12-05 21:11:48.276616008 +0100
> @@ -835,7 +835,9 @@ EXTRA_TARGET_FLAGS = \
>  	'AS=$(COMPILER_AS_FOR_TARGET)' \
>  	'CC=$$(CC_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CFLAGS=$$(CFLAGS_FOR_TARGET)' \
> -	'CXX=$$(CXX_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> +	'CXX=$$(CXX_FOR_TARGET) -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
> +	 -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs \
> +	 $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CXXFLAGS=$$(CXXFLAGS_FOR_TARGET)' \
>  	'DLLTOOL=$$(DLLTOOL_FOR_TARGET)' \
>  	'GCJ=$$(GCJ_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> @@ -853,6 +855,8 @@ EXTRA_TARGET_FLAGS = \
>  	'WINDRES=$$(WINDRES_FOR_TARGET)' \
>  	'WINDMC=$$(WINDMC_FOR_TARGET)' \
>  	'XGCC_FLAGS_FOR_TARGET=$(XGCC_FLAGS_FOR_TARGET)' \
> +	'STAGE1_LDFLAGS=$$(POSTSTAGE1_LDFLAGS)' \
> +	'STAGE1_LIBS=$$(POSTSTAGE1_LIBS)' \
>  	"TFLAGS=$$TFLAGS"
>  
>  TARGET_FLAGS_TO_PASS = $(BASE_FLAGS_TO_PASS) $(EXTRA_TARGET_FLAGS)
> 
> 	Jakub
> 

If it also works for --disable-bootstrap, this is okay.

Paolo

WARNING: multiple messages have this Message-ID
From: Paolo Bonzini <bonzini@gnu.org>
To: gcc-patches@gcc.gnu.org
Cc: gcc@gcc.gnu.org
Subject: Re: [patch, build] Restore bootstrap in building libcc1 on darwin
Date: Sat, 13 Dec 2014 11:26:00 -0000	[thread overview]
Message-ID: <548C1A51.9020301@gnu.org> (raw)
Message-ID: <20141213112600.4P4USoRiODK8Vd7fahZPWskv-c5R5dyUR3mlXsWnUTw@z> (raw)
In-Reply-To: <20141205224751.GS1923@tucnak.redhat.com>



On 05/12/2014 23:47, Jakub Jelinek wrote:
> On Fri, Dec 05, 2014 at 11:34:28PM +0100, Dominique Dhumieres wrote:
>>> As I've tried to explain, that is IMHO wrong though.
>>> If what you are after is the -B stuff too, then perhaps:
>>> ...
>>
>> Sorry but it does not work:
> 
> Sorry, make that (just removed 4x ' in each file):
> 
> 2014-12-05  Jakub Jelinek  <jakub@redhat.com>
> 
> 	PR bootstrap/64023
> 	* Makefile.tpl (EXTRA_TARGET_FLAGS): Set STAGE1_LDFLAGS
> 	to POSTSTAGE1_LDFLAGS and STAGE1_LIBS to POSTSTAGE1_LIBS.
> 	Add -B to libstdc++-v3/src/.libs and libstdc++-v3/libsupc++/.libs
> 	to CXX.
> 	* Makefile.in: Regenerated.
> 
> --- Makefile.tpl.jj	2014-11-12 09:31:59.000000000 +0100
> +++ Makefile.tpl	2014-12-05 21:12:21.486031062 +0100
> @@ -641,7 +641,9 @@ EXTRA_TARGET_FLAGS = \
>  	'AS=$(COMPILER_AS_FOR_TARGET)' \
>  	'CC=$$(CC_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CFLAGS=$$(CFLAGS_FOR_TARGET)' \
> -	'CXX=$$(CXX_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> +	'CXX=$$(CXX_FOR_TARGET) -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
> +	 -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs \
> +	 $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CXXFLAGS=$$(CXXFLAGS_FOR_TARGET)' \
>  	'DLLTOOL=$$(DLLTOOL_FOR_TARGET)' \
>  	'GCJ=$$(GCJ_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> @@ -659,6 +661,8 @@ EXTRA_TARGET_FLAGS = \
>  	'WINDRES=$$(WINDRES_FOR_TARGET)' \
>  	'WINDMC=$$(WINDMC_FOR_TARGET)' \
>  	'XGCC_FLAGS_FOR_TARGET=$(XGCC_FLAGS_FOR_TARGET)' \
> +	'STAGE1_LDFLAGS=$$(POSTSTAGE1_LDFLAGS)' \
> +	'STAGE1_LIBS=$$(POSTSTAGE1_LIBS)' \
>  	"TFLAGS=$$TFLAGS"
>  
>  TARGET_FLAGS_TO_PASS = $(BASE_FLAGS_TO_PASS) $(EXTRA_TARGET_FLAGS)
> --- Makefile.in.jj	2014-11-28 14:40:52.000000000 +0100
> +++ Makefile.in	2014-12-05 21:11:48.276616008 +0100
> @@ -835,7 +835,9 @@ EXTRA_TARGET_FLAGS = \
>  	'AS=$(COMPILER_AS_FOR_TARGET)' \
>  	'CC=$$(CC_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CFLAGS=$$(CFLAGS_FOR_TARGET)' \
> -	'CXX=$$(CXX_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> +	'CXX=$$(CXX_FOR_TARGET) -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
> +	 -B$$r/$$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs \
> +	 $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
>  	'CXXFLAGS=$$(CXXFLAGS_FOR_TARGET)' \
>  	'DLLTOOL=$$(DLLTOOL_FOR_TARGET)' \
>  	'GCJ=$$(GCJ_FOR_TARGET) $$(XGCC_FLAGS_FOR_TARGET) $$(TFLAGS)' \
> @@ -853,6 +855,8 @@ EXTRA_TARGET_FLAGS = \
>  	'WINDRES=$$(WINDRES_FOR_TARGET)' \
>  	'WINDMC=$$(WINDMC_FOR_TARGET)' \
>  	'XGCC_FLAGS_FOR_TARGET=$(XGCC_FLAGS_FOR_TARGET)' \
> +	'STAGE1_LDFLAGS=$$(POSTSTAGE1_LDFLAGS)' \
> +	'STAGE1_LIBS=$$(POSTSTAGE1_LIBS)' \
>  	"TFLAGS=$$TFLAGS"
>  
>  TARGET_FLAGS_TO_PASS = $(BASE_FLAGS_TO_PASS) $(EXTRA_TARGET_FLAGS)
> 
> 	Jakub
> 

If it also works for --disable-bootstrap, this is okay.

Paolo

  parent reply	other threads:[~2014-12-13 10:52 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-05 22:34 Dominique Dhumieres
2014-12-05 22:41 ` Jeff Law
2014-12-05 22:46   ` Phil Muldoon
2014-12-06 11:16   ` Iain Sandoe
2014-12-05 22:48 ` Jakub Jelinek
2014-12-06  0:50   ` Dominique d'Humières
2014-12-09 17:08     ` Dominique d'Humières
2014-12-13 10:52   ` Paolo Bonzini [this message]
2014-12-13 11:26     ` Paolo Bonzini
  -- strict thread matches above, loose matches on Subject: below --
2014-12-05 19:12 Dominique Dhumieres
2014-12-05 20:16 ` Jakub Jelinek
     [not found] <94381BB6-563C-44EE-910E-9569410F2AB5@gmail.com>
2014-11-24  2:28 ` FX
2014-11-25 20:59   ` Mike Stump
2014-11-25 21:37     ` Phil Muldoon
2014-11-25 23:27       ` Jeff Law
2014-12-04  9:12   ` FX
2014-12-04 13:32     ` Rainer Orth
2014-12-04 13:37       ` FX
2014-12-04 13:47         ` Rainer Orth
2014-12-04 13:43       ` Iain Sandoe
2014-12-04 13:47         ` Richard Biener
2014-12-04 13:48           ` Richard Biener
2014-12-04 13:53             ` Richard Biener
2014-12-04 13:52         ` FX
2014-12-04 13:54           ` Richard Biener
2014-12-04 15:24             ` FX
2014-12-04 15:36               ` Iain Sandoe
2015-01-26 15:02           ` Rainer Orth
2015-01-26 15:06             ` Iain Sandoe
2015-02-01 16:42             ` Iain Sandoe
2015-02-09 22:47               ` Jeff Law
2015-02-11 14:38                 ` Iain Sandoe
2015-03-08 17:38                   ` Iain Sandoe
2015-03-08 17:48                     ` Ian Lance Taylor
2015-03-09 18:43                     ` Mike Stump
2014-12-05 16:35   ` Jakub Jelinek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=548C1A51.9020301@gnu.org \
    --to=bonzini@gnu.org \
    --cc=dominiq@lps.ens.fr \
    --cc=fxcoudert@gmail.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gcc@gcc.gnu.org \
    --cc=iain@codesourcery.com \
    --cc=jakub@redhat.com \
    --cc=pmuldoon@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).