On Tue, 6 Dec 2011, Han Shen wrote:
+gcc_gxx_include_dir_add_sysroot=0
+if test ${with_sysroot+set} = set; then :
+ gcc_gxx_without_sysroot=`expr ${gcc_gxx_include_dir} :
${with_sysroot}'\(.*\)'`
+ if test ${gcc_gxx_without_sysroot}; then :
No need for the : after then in two places
Hi, Joseph, thanks!
Yeah, I see where the problem is - I posted all these
(patchset,Changelog, rationale and previous gcc-patches discussion.)
on http://codereview.appspot.com/5394041
So in addition to that, I include them all here
==
This is a follow up for issue
On Fri, 18 Nov 2011, Han Shen wrote:
Hi, Joseph, thanks!
ChangeLog entries added to the issue description.
ChangeLog
* Makefile.in (GPLUSPLUS_INCLUDE_DIR_ADD_SYSROOT): add a macro
definition to compile command.
* cppdefault.c (GPLUSPLUS_INCLUDE_DIR_ADD_SYSROOT):
On Tue, 15 Nov 2011, Han Shen wrote:
2011-11-15 Han Shen shen...@google.com
* gcc/Makefile.in:
* gcc/configure:
* gcc/cppdefault.c:
You need to include the full ChangeLog entries with your patch. Note that
paths in ChangeLogs are relative to the directory with the
Hi, Joseph, thanks!
ChangeLog entries added to the issue description.
ChangeLog
* Makefile.in (GPLUSPLUS_INCLUDE_DIR_ADD_SYSROOT): add a macro
definition to compile command.
* cppdefault.c (GPLUSPLUS_INCLUDE_DIR_ADD_SYSROOT): replace hard
coded add_sysroot field
2011-11-15 Han Shen shen...@google.com
* gcc/Makefile.in:
* gcc/configure:
* gcc/cppdefault.c:
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index ae4f4da..0a05783 100644
--- a/gcc/Makefile.in
+++ b/gcc/Makefile.in
@@ -615,6 +615,7 @@ gcc_tooldir = @gcc_tooldir@
