[ adding gcc-patches ] * Gerald Pfeifer wrote on Sun, Feb 17, 2008 at 02:46:45PM CET: > On Sun, 17 Feb 2008, Ralf Wildenhues wrote: > > I see two possibilities: revert above patch, and list texinfo 4.11 as > > prerequisite for building the pdf/dvi > > I'm not in favor of making texinfo 4.11 a requirement. As far as I can > see it was released less than half a year ago and even recent releases of > popular systems such as openSUSE 10.3 or FreeBSD 6.3, both released a few > months ago do not have this yet, though admittedly Fedora 8 does.
Note that it would be a requirement for `make dvi pdf' only. > No objection to make this a requirement for MinGW only, of course. That doesn't help, though, as the 4.11 requirement would be the alternative fix for PR 35148. So, here's another alternative: fix the MinGW issue only, while not regressing on any of the other reported bug (famous last words): One way for path translation on MinGW would be to use CMD //c echo "@set srcdir" "$(abs_srcdir)" as in the patch below. I need somebody to test this patch on MinGW, all I've done is ensure build_file_translate expands to empty on i686-pc-linux-gnu. Thanks, Ralf gcc/ChangeLog: 2008-02-17 Ralf Wildenhues <[EMAIL PROTECTED]> PR bootstrap/35218 * Makefile.in (build_file_translate): New. * config.build (build_file_translate): Set to `CMD //c' on MinGW. * configure.ac (build_file_translate): Substitute it. * configure: Regenerate. diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 9c91fb5..022208a 100644 --- a/gcc/Makefile.in +++ b/gcc/Makefile.in @@ -253,6 +253,8 @@ POD2MAN = pod2man --center="GNU" --release="gcc-$(version)" # do not correctly set the timestamp due to buggy versions of `utime' # in the kernel. So, we use `echo' instead. STAMP = echo timestamp > +# Translate absolute file names to w32 paths. +build_file_translate = @build_file_translate@ # Make sure the $(MAKE) variable is defined. @SET_MAKE@ @@ -3653,7 +3655,7 @@ gcc-vers.texi: $(BASEVER) $(DEVPHASE) then echo "@set DEVELOPMENT"; \ else echo "@clear DEVELOPMENT"; \ fi) > [EMAIL PROTECTED] - echo "@set srcdir $(abs_srcdir)" >> [EMAIL PROTECTED] + $(build_file_translate) echo "@set srcdir" "$(abs_srcdir)" >> [EMAIL PROTECTED] if [ -n "$(PKGVERSION)" ]; then \ echo "@set VERSION_PACKAGE $(PKGVERSION)" >> [EMAIL PROTECTED]; \ fi diff --git a/gcc/config.build b/gcc/config.build index 4f5421b..62486db 100644 --- a/gcc/config.build +++ b/gcc/config.build @@ -47,6 +47,7 @@ build_xm_file= build_xm_defines= build_exeext= build_install_headers_dir=install-headers-tar +build_file_translate= # System-specific settings. case $build in @@ -79,6 +80,7 @@ case $build in i[34567]86-*-mingw32* | x86_64-*-mingw*) build_xm_file=i386/xm-mingw32.h build_exeext=.exe + build_file_translate="CMD //C" ;; i[34567]86-pc-msdosdjgpp*) build_xm_file=i386/xm-djgpp.h diff --git a/gcc/configure.ac b/gcc/configure.ac index f3bc7b3..bb7c056 100644 --- a/gcc/configure.ac +++ b/gcc/configure.ac @@ -3727,6 +3727,7 @@ AC_SUBST(build_install_headers_dir) AC_SUBST(build_xm_file_list) AC_SUBST(build_xm_include_list) AC_SUBST(build_xm_defines) +AC_SUBST(build_file_translate) AC_SUBST(check_languages) AC_SUBST(cpp_install_dir) AC_SUBST(xmake_file)