Skip to content
This repository has been archived by the owner on Feb 11, 2021. It is now read-only.

Commit

Permalink
Fixed bug 1972 - Changeset 7379 (b27c778a2bdb) breaks make process wi…
Browse files Browse the repository at this point in the history
…th msys+mingw (make 3.82.90)

Andreas

With the patch applied, make is not able to find the rule for Makefile.in anymore. Removing the patch resolves the issue.

The path is in fact correct (in my case: /c/external/SDL64/SDL). But it seems the windows build of GNU Make doesn't work well with pathnames in rules. Both the dependencies in "$(srcdir)/configure: $(srcdir)/configure.in" and "Makefile: $(srcdir)/Makefile.in" will cause rules not to be found when srcdir is defined.

The same problem occurs if the patch is removed and I supply configure with a srcdir manually.
  • Loading branch information
slouken committed Aug 4, 2013
1 parent 4bb9864 commit 70bf97e
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 2 deletions.
9 changes: 8 additions & 1 deletion configure
Expand Up @@ -15791,7 +15791,14 @@ else
fi


srcdir=`cd $srcdir && pwd`
case "$host" in
*-*-mingw32*)
# Except on msys, where make can't handle full pathnames (bug 1972)
;;
*)
srcdir=`cd $srcdir && pwd`
;;
esac

INCLUDE="-I$srcdir/include"
if test x$srcdir != x.; then
Expand Down
9 changes: 8 additions & 1 deletion configure.in
Expand Up @@ -58,7 +58,14 @@ AC_PROG_MAKE_SET
AC_CHECK_TOOL(WINDRES, [windres], [:])

dnl Make sure that srcdir is a full pathname
srcdir=`cd $srcdir && pwd`
case "$host" in
*-*-mingw32*)
# Except on msys, where make can't handle full pathnames (bug 1972)
;;
*)
srcdir=`cd $srcdir && pwd`
;;
esac

dnl Set up the compiler and linker flags
INCLUDE="-I$srcdir/include"
Expand Down

0 comments on commit 70bf97e

Please sign in to comment.