PATH_SEPARATOR attempted redefined in gcc-4.6 snapshot on certain targets

Pedro Alves pedro@codesourcery.com
Mon Apr 26 21:02:00 GMT 2010


On Monday 26 April 2010 14:50:48, Ian Lance Taylor wrote:
> Pedro, the incomplete bug report appears to be due to your change to
> include/filenames.h.  Tim, what configure options are you using to
> build?
> 
> Pedro, by the way, your ChangeLog entry for the include directory does
> not mention that you define PATH_SEPARATOR.  Would you mind fixing
> that while you're at it?

Whoops, sure, sorry.  Though since I don't have gcc svn write access,
it takes an extra hoop to fix.  :-)

Index: src/include/ChangeLog
===================================================================
--- src.orig/include/ChangeLog  2010-04-26 15:15:59.000000000 +0100
+++ src/include/ChangeLog       2010-04-26 15:16:02.000000000 +0100
@@ -3,6 +3,7 @@
        * filenames.h (IS_DIR_SEPARATOR_1): Rename from IS_DIR_SEPARATOR,
        always define it independently of host, add `dos_based' parameter,
        and handle it.
+       (PATH_SEPARATOR): Define.
        (HAS_DRIVE_SPEC_1): Rename from HAS_DRIVE_SPEC, always define it
        independently of host, add `dos_based' parameter, and handle it.
        (IS_ABSOLUTE_PATH_1): Rename from IS_ABSOLUTE_PATH, always define

And, this is embarassing, I don't actually need PATH_SEPARATOR in
filenames.h in the end.  It was only required in an earlier version
of my gdb patch, but that version never made it to the public.  Sorry
I missed this.

I'll post the obvious patch removing it at gcc-patches@.

-- 
Pedro Alves



More information about the Gcc-help mailing list