public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH 1/1] Remove redundant definition of srcrootpre
@ 2017-06-05  7:30 coypu
  2017-06-06 17:45 ` Jeff Law
  0 siblings, 1 reply; 3+ messages in thread
From: coypu @ 2017-06-05  7:30 UTC (permalink / raw)
  To: gcc-patches

This script has the only occurrence of it, and in this line
it defines and exports it.

---
 config-ml.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/config-ml.in b/config-ml.in
index 47f153350..58c80a35c 100644
--- a/config-ml.in
+++ b/config-ml.in
@@ -493,7 +493,6 @@ multi-do:
 	  true; \
 	else \
 	  rootpre=`${PWD_COMMAND}`/; export rootpre; \
-	  srcrootpre=`cd $(srcdir); ${PWD_COMMAND}`/; export srcrootpre; \
 	  lib=`echo "$${rootpre}" | sed -e 's,^.*/\([^/][^/]*\)/$$,\1,'`; \
 	  compiler="$(CC)"; \
 	  for i in `$${compiler} --print-multi-lib 2>/dev/null`; do \
-- 
2.12.2

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] Remove redundant definition of srcrootpre
  2017-06-05  7:30 [PATCH 1/1] Remove redundant definition of srcrootpre coypu
@ 2017-06-06 17:45 ` Jeff Law
  2017-06-08  9:39   ` coypu
  0 siblings, 1 reply; 3+ messages in thread
From: Jeff Law @ 2017-06-06 17:45 UTC (permalink / raw)
  To: coypu, gcc-patches

On 06/05/2017 01:30 AM, coypu wrote:
> This script has the only occurrence of it, and in this line
> it defines and exports it.
> 
> ---
>  config-ml.in | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/config-ml.in b/config-ml.in
> index 47f153350..58c80a35c 100644
> --- a/config-ml.in
> +++ b/config-ml.in
> @@ -493,7 +493,6 @@ multi-do:
>  	  true; \
>  	else \
>  	  rootpre=`${PWD_COMMAND}`/; export rootpre; \
> -	  srcrootpre=`cd $(srcdir); ${PWD_COMMAND}`/; export srcrootpre; \
>  	  lib=`echo "$${rootpre}" | sed -e 's,^.*/\([^/][^/]*\)/$$,\1,'`; \
>  	  compiler="$(CC)"; \
>  	  for i in `$${compiler} --print-multi-lib 2>/dev/null`; do \
> 
I think this is used outside GCC (newlib in particular).  It may be a
remnant of when folks used to build everything in a single unified tree
for crosses.

jeff

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] Remove redundant definition of srcrootpre
  2017-06-06 17:45 ` Jeff Law
@ 2017-06-08  9:39   ` coypu
  0 siblings, 0 replies; 3+ messages in thread
From: coypu @ 2017-06-08  9:39 UTC (permalink / raw)
  To: Jeff Law; +Cc: gcc-patches

I paid extra attention to it because it appeared in the last line of a
build failure likely caused by shell tools differences. cd rarely outputs
the new directory for me.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-06-08  9:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-05  7:30 [PATCH 1/1] Remove redundant definition of srcrootpre coypu
2017-06-06 17:45 ` Jeff Law
2017-06-08  9:39   ` coypu

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).