Alexandre Julliard : makefiles: Don't use Make. rules in the top-level Makefile.

Alexandre Julliard julliard at winehq.org
Mon Dec 16 13:02:31 CST 2013


Module: wine
Branch: master
Commit: a8b633b1a194d12a946f39cbf4cfed46953e5360
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=a8b633b1a194d12a946f39cbf4cfed46953e5360

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Sun Dec 15 10:38:25 2013 +0100

makefiles: Don't use Make.rules in the top-level Makefile.

---

 Make.rules.in        |    1 -
 Make.vars.in         |    1 +
 Makefile.in          |   14 ++++++++------
 configure            |    4 ++--
 configure.ac         |    4 ++--
 tools/make_makefiles |    1 +
 6 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/Make.rules.in b/Make.rules.in
index 27e5667..5948180 100644
--- a/Make.rules.in
+++ b/Make.rules.in
@@ -19,7 +19,6 @@ RCFLAGS      = --nostdinc $(PORCFLAGS) $(TARGETFLAGS) $(DEFS) $(EXTRARCFLAGS)
 IDL_GEN_C_SRCS  = $(IDL_C_SRCS:.idl=_c.c) $(IDL_I_SRCS:.idl=_i.c) \
                   $(IDL_P_SRCS:.idl=_p.c) $(IDL_S_SRCS:.idl=_s.c)
 
-CLEAN_FILES   = *.o *.a *.so *.res *.fake *.ok *.tab.[ch] *.yy.c
 CLEAN_TARGETS = $(MODULE) $(IMPORTLIB:%=lib%. at IMPLIBEXT@) \
                 $(TESTMODULE) $(TESTMODULE_STRIPPED) $(CROSSTESTMODULE)
 
diff --git a/Make.vars.in b/Make.vars.in
index ac74f1f..7b6e265 100644
--- a/Make.vars.in
+++ b/Make.vars.in
@@ -76,6 +76,7 @@ INSTALL         = @INSTALL@ $(INSTALL_FLAGS)
 INSTALL_PROGRAM = @INSTALL_PROGRAM@ $(INSTALL_PROGRAM_FLAGS)
 INSTALL_SCRIPT  = @INSTALL_SCRIPT@ $(INSTALL_SCRIPT_FLAGS)
 INSTALL_DATA    = @INSTALL_DATA@ $(INSTALL_DATA_FLAGS)
+CLEAN_FILES     = *.o *.a *.so *.res *.fake *.ok *.tab.[ch] *.yy.c
 prog_manext     = 1
 api_manext      = 3w
 conf_manext     = 5
diff --git a/Makefile.in b/Makefile.in
index c2bb1ed..c7889d0 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -28,8 +28,6 @@ INSTALLDIRS = \
 all: wine
 	@echo "Wine build complete."
 
- at MAKE_RULES@
-
 # Rules for re-running configure
 
 config.status: $(srcdir)/configure
@@ -41,15 +39,19 @@ include/stamp-h: $(srcdir)/include/config.h.in config.status
 
 # Rules for cleaning
 
-.PHONY: __clean__
+.PHONY: clean distclean __clean__
+
 clean:: __clean__
 distclean:: clean
 	$(RM) config.* configure.lineno TAGS tags include/config.h include/stamp-h Makefile Make.tmp
 	$(RM) -r autom4te.cache
 
-# Rules for uninstalling
+# Rules for installing
+
+$(INSTALLDIRS):
+	$(MKDIR_P) -m 755 $@
 
-.PHONY: __uninstall__
+.PHONY: install install-lib install-dev uninstall __uninstall__
 uninstall:: __uninstall__
 	-rmdir $(DESTDIR)$(datadir)/wine $(DESTDIR)$(fakedlldir) $(DESTDIR)$(dlldir)
 
@@ -58,7 +60,7 @@ uninstall:: __uninstall__
 # dependencies needed to build any dll or program
 __tooldeps__: libs/port libs/wine libs/wpp
 __builddeps__: __tooldeps__ include
-.PHONY: check test testclean crosstest __tooldeps__ __builddeps__
+.PHONY: depend check test testclean crosstest __tooldeps__ __builddeps__
 
 loader server: libs/port libs/wine tools
 fonts: tools
diff --git a/configure b/configure
index 8fde226..e604301 100755
--- a/configure
+++ b/configure
@@ -16681,7 +16681,7 @@ wine_fn_append_rule "
 .INIT: Makefile
 .MAKEFILEDEPS:
 all: Makefile
-Makefile: $srcdir/Makefile.in $srcdir/Make.vars.in Make.rules config.status
+Makefile: $srcdir/Makefile.in $srcdir/Make.vars.in config.status
 	@./config.status Make.tmp Makefile"
 
 if test "x$enable_fonts" != xno; then
@@ -17525,7 +17525,7 @@ then
     ac_config_commands="$ac_config_commands tools"
 
     wine_fn_append_rule "tools/makedep$ac_exeext: $srcdir/tools/makedep.c include/config.h
-	\$(CC) -Iinclude -I$srcdir/include \$(ALLCFLAGS) -o \$@ $srcdir/tools/makedep.c \$(LDFLAGS)
+	\$(CC) -Iinclude -I$srcdir/include -D__WINESRC__ \$(EXTRACFLAGS) \$(CPPFLAGS) \$(CFLAGS) -o \$@ $srcdir/tools/makedep.c \$(LDFLAGS)
 clean:: __clean__
 	\$(RM) tools/makedep$ac_exeext"
 else
diff --git a/configure.ac b/configure.ac
index 34b7a52..fc6cdde 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2664,7 +2664,7 @@ WINE_APPEND_RULE([
 .INIT: Makefile
 .MAKEFILEDEPS:
 all: Makefile
-Makefile: $srcdir/Makefile.in $srcdir/Make.vars.in Make.rules config.status
+Makefile: $srcdir/Makefile.in $srcdir/Make.vars.in config.status
 	@./config.status Make.tmp Makefile])
 
 WINE_CONFIG_SYMLINK(fonts,fonts,[marlett.ttf,symbol.ttf,tahoma.ttf,tahomabd.ttf],enable_fonts)
@@ -3502,7 +3502,7 @@ then
     AC_CONFIG_COMMANDS([tools],[AS_MKDIR_P(tools)])
     WINE_APPEND_RULE(
 [tools/makedep$ac_exeext: $srcdir/tools/makedep.c include/config.h
-	\$(CC) -Iinclude -I$srcdir/include \$(ALLCFLAGS) -o \$@ $srcdir/tools/makedep.c \$(LDFLAGS)
+	\$(CC) -Iinclude -I$srcdir/include -D__WINESRC__ \$(EXTRACFLAGS) \$(CPPFLAGS) \$(CFLAGS) -o \$@ $srcdir/tools/makedep.c \$(LDFLAGS)
 clean:: __clean__
 	\$(RM) tools/makedep$ac_exeext])
 else
diff --git a/tools/make_makefiles b/tools/make_makefiles
index 13106e7..34fef85 100755
--- a/tools/make_makefiles
+++ b/tools/make_makefiles
@@ -543,6 +543,7 @@ sub update_makefiles(@)
     foreach my $file (sort @_)
     {
         my %make = %{$makefiles{$file}};
+        next unless defined $make{"=rules"};
         my $rules = $make{"=rules"};
         my $args = "";
         my $is_win16 = $make{"MODULE"} && ($make{"MODULE"} =~ /16$/ || $modules16{$make{"MODULE"}});




More information about the wine-cvs mailing list