# HG changeset patch # User Rik # Date 1263278288 28800 # Node ID 82d47b70642404151336dba0669cbacfddb4f4de # Parent 81e88250bf422472e868c965060d07be5eeecd81 Remove unnecessary uses of simple_move_if_change_rule Rule prevents make from updating timestamps and should only be used where definite savings from reduced compilations outweight the commands that are executed every time make is invoked. diff --git a/liboctave/ChangeLog b/liboctave/ChangeLog --- a/liboctave/ChangeLog +++ b/liboctave/ChangeLog @@ -1,3 +1,8 @@ +2010-01-11 Rik + + * Makefile.am: Remove unnecessary use of simple_move_if_change_rule to allow + Make to update timestamps + 2010-01-08 Jaroslav Hajek * Array.h (Array::crefT): New helper typedef. diff --git a/liboctave/Makefile.am b/liboctave/Makefile.am --- a/liboctave/Makefile.am +++ b/liboctave/Makefile.am @@ -530,7 +530,7 @@ $(OPT_INC) : %.h : %.in $(top_srcdir)/mk-opts.pl @echo making $@ from $< @$(PERL) $(top_srcdir)/mk-opts.pl --opt-class-header $< > $@-t - $(simple_move_if_change_rule) + mv $@-t $@ $(VX_OP_INC) $(VX_OP_SRC) : $(srcdir)/mk-ops.awk vx-ops $(AWK) -f $(srcdir)/mk-ops.awk prefix=vx $(srcdir)/vx-ops @@ -543,7 +543,7 @@ mx-ops.h : $(srcdir)/mk-ops.awk mx-ops $(AWK) -f $(srcdir)/mk-ops.awk prefix=mx make_inclusive_header=mx-ops.h $(srcdir)/mx-ops > $@-t - $(simple_move_if_change_rule) + mv $@-t $@ DISTCLEANFILES = $(BUILT_INCS) $(BUILT_LIBOCTAVE_CXX_SOURCES) diff --git a/scripts/ChangeLog b/scripts/ChangeLog --- a/scripts/ChangeLog +++ b/scripts/ChangeLog @@ -1,3 +1,8 @@ +2010-01-11 Rik + + * Makefile.am: Update DOCSTRINGS rules to avoid simple_move_if_change_rule + and allow Make to update timestamps. + 2010-01-11 Jaroslav Hajek * plot/private/__scatter__.m: Use patch objects instead of lines, diff --git a/scripts/Makefile.am b/scripts/Makefile.am --- a/scripts/Makefile.am +++ b/scripts/Makefile.am @@ -74,7 +74,7 @@ FCN_FILES_IN = $(GEN_FCN_FILES:.m=.in) -all-local: $(GEN_FCN_FILES) $(PKG_ADD_FILES) DOCSTRINGS +all-local: $(GEN_FCN_FILES) $(PKG_ADD_FILES) .DOCSTRINGS octave_dirstamp = $(am__leading_dot)dirstamp @@ -317,11 +317,15 @@ gethelp_SOURCES = gethelp.cc -DOCSTRINGS: $(FCN_FILES) $(GEN_FCN_FILES) mkdoc $(gethelp_SOURCES) +.DOCSTRINGS: $(FCN_FILES) $(GEN_FCN_FILES) mkdoc $(gethelp_SOURCES) Makefile @$(MAKE) $(AM_MAKEFLAGS) gethelp$(BUILD_EXEEXT) - $(cp_update_rule) - $(srcdir)/mkdoc "$(srcdir)" $(FCN_FILES) -- $(GEN_FCN_FILES) > $@-t - $(simple_move_if_change_rule) + if [ "x$(srcdir)" != "x." ] && [ -f $(srcdir)/DOCSTRINGS ] && [ ! -f DOCSTRINGS ]; then \ + cp $(srcdir)/DOCSTRINGS DOCSTRINGS; \ + touch -r $(srcdir)/DOCSTRINGS DOCSTRINGS; \ + fi + $(srcdir)/mkdoc "$(srcdir)" $(FCN_FILES) -- $(GEN_FCN_FILES) > $@ + $(top_srcdir)/move-if-change $@ DOCSTRINGS + touch $@ $(GEN_FCN_FILES) : %.m : %.in Makefile $(do_subst_config_vals) diff --git a/src/ChangeLog b/src/ChangeLog --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,3 +1,8 @@ +2010-01-11 Rik + + * Makefile.am: Update DOCSTRINGS rules to avoid simple_move_if_change_rule + and allow Make to update timestamps. + 2010-01-11 John W. Eaton * DLD-FUNCTIONS/urlwrite.cc (curl_handle::lasterror): diff --git a/src/Makefile.am b/src/Makefile.am --- a/src/Makefile.am +++ b/src/Makefile.am @@ -591,7 +591,7 @@ octave_LDADD = $(OCTAVE_LIBS) -all-local: $(OCT_FILES) DLD-FUNCTIONS/PKG_ADD DOCSTRINGS +all-local: $(OCT_FILES) DLD-FUNCTIONS/PKG_ADD .DOCSTRINGS lex.lo lex.o oct-parse.lo oct-parse.o: \ AM_CXXFLAGS := $(filter-out -Wold-style-cast, $(AM_CXXFLAGS)) @@ -631,10 +631,14 @@ $(srcdir)/mk-pkg-add $(DLD_DYNAMIC_DEF_FILES) > $@-t mv $@-t $@ -DOCSTRINGS: gendoc$(BUILD_EXEEXT) - $(cp_update_rule) - ./gendoc > $@-t - $(simple_move_if_change_rule) +.DOCSTRINGS: gendoc$(BUILD_EXEEXT) + if [ "x$(srcdir)" != "x." ] && [ -f $(srcdir)/DOCSTRINGS ] && [ ! -f DOCSTRINGS ]; then \ + cp $(srcdir)/DOCSTRINGS DOCSTRINGS; \ + touch -r $(srcdir)/DOCSTRINGS DOCSTRINGS; \ + fi + ./gendoc > $@ + $(top_srcdir)/move-if-change $@ DOCSTRINGS + touch $@ doc-files: $(DEF_FILES) echo $(DEF_FILES) > $@-t