This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: bootstrap broken on mingw
- From: Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>
- To: Gerald Pfeifer <gerald at pfeifer dot com>
- Cc: FX Coudert <fxcoudert at gmail dot com>, Richard Guenther <richard dot guenther at gmail dot com>, Joerg Wunsch <j at uriah dot heep dot sax dot de>, GCC Development <gcc at gcc dot gnu dot org>, "Weddington, Eric" <eweddington at cso dot atmel dot com>, Denis Chertykov <chertykov at gmail dot com>, Anatoly Sokolov <aesok at post dot ru>, "Lamo, Jo Inge" <jo-inge dot lamo at atmel dot com>, gcc-patches at gcc dot gnu dot org
- Date: Sun, 17 Feb 2008 15:10:41 +0100
- Subject: Re: bootstrap broken on mingw
- References: <6E32C92C-4934-4B9D-8A9D-014BFC395383@gmail.com> <84fc9c000802170351v36c3a73ey560e127978edf31f@mail.gmail.com> <3D984872-3514-40F7-A03C-9F3D44042C1B@gmail.com> <20080217131812.GA22680@ins.uni-bonn.de> <alpine.LSU.1.00.0802171430210.32184@pulcherrima.dbai.tuwien.ac.at>
[ 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 <Ralf.Wildenhues@gmx.de>
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) > $@T
- echo "@set srcdir $(abs_srcdir)" >> $@T
+ $(build_file_translate) echo "@set srcdir" "$(abs_srcdir)" >> $@T
if [ -n "$(PKGVERSION)" ]; then \
echo "@set VERSION_PACKAGE $(PKGVERSION)" >> $@T; \
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)