Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: > Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx> > --- This patch is wrong, I think. The canonical example of order-only is something like http://www.kolpackov.net/pipermail/notes/2004-January/000001.html where you want to make sure an output directory already exists, but you do not want to list that as prerequiste _input_ to produce output from the source. G-V-F has real information that is used to affect the output, which is quite a different animal. Try applying this 6-patch series, "make clean; make git.spec", and then "git checkout HEAD~2" to come back to the commit for this patch. And then try running "make git.spec". I think Version line will not change and still claim you are building a version with the 6-patch series fully applied. After a quick scan of the other 5 patches in the series, they looked all sensible, and I do not think any of them depends on the change in this patch, so I think we can safely drop this one. Administrivia: as I said in the whats cooking message, I'll be taking things slow til the beginning of next month, so if your (not limited to Felipe) messages are not answered by me, it merely means they were not yet read by me (as opposed to dismissed as not worth reading without responding). > Makefile | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Makefile b/Makefile > index 28b6117..97ff848 100644 > --- a/Makefile > +++ b/Makefile > @@ -1787,7 +1787,7 @@ perl/PM.stamp: FORCE > perl/perl.mak: GIT-CFLAGS GIT-PREFIX perl/Makefile perl/Makefile.PL > $(QUIET_SUBDIR0)perl $(QUIET_SUBDIR1) PERL_PATH='$(PERL_PATH_SQ)' prefix='$(prefix_SQ)' $(@F) > > -$(patsubst %.perl,%,$(SCRIPT_PERL)): % : %.perl GIT-VERSION-FILE > +$(patsubst %.perl,%,$(SCRIPT_PERL)): % : %.perl | GIT-VERSION-FILE > $(QUIET_GEN)$(RM) $@ $@+ && \ > INSTLIBDIR=`MAKEFLAGS= $(MAKE) -C perl -s --no-print-directory instlibdir` && \ > sed -e '1{' \ > @@ -1850,7 +1850,7 @@ CONFIGURE_RECIPE = $(RM) configure configure.ac+ && \ > autoconf -o configure configure.ac+ && \ > $(RM) configure.ac+ > > -configure: configure.ac GIT-VERSION-FILE > +configure: configure.ac | GIT-VERSION-FILE > $(QUIET_GEN)$(CONFIGURE_RECIPE) > > ifdef AUTOCONFIGURED > @@ -2415,7 +2415,7 @@ quick-install-html: > > ### Maintainer's dist rules > > -git.spec: git.spec.in GIT-VERSION-FILE > +git.spec: git.spec.in | GIT-VERSION-FILE > sed -e 's/@@VERSION@@/$(GIT_VERSION)/g' < $< > $@+ > mv $@+ $@ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html