git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Stop making git-merge-recur
@ 2006-10-02  2:45 A Large Angry SCM
  2006-10-02  2:49 ` A Large Angry SCM
  2006-10-02  4:41 ` Junio C Hamano
  0 siblings, 2 replies; 4+ messages in thread
From: A Large Angry SCM @ 2006-10-02  2:45 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Git-merge-recur is no longer installed and make clean no longer
removes it, so creating making it.


Signed-off-by: A Large Angry SCM <gitzilla@gmail.com>
---

 Makefile |    8 ++------
 1 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile
index dd33158..37bd59d 100644
--- a/Makefile
+++ b/Makefile
@@ -586,8 +586,7 @@ export prefix TAR INSTALL DESTDIR SHELL_
 
 ### Build rules
 
-all: $(ALL_PROGRAMS) $(BUILT_INS) git$X gitk gitweb/gitweb.cgi \
-	git-merge-recur$X
+all: $(ALL_PROGRAMS) $(BUILT_INS) git$X gitk gitweb/gitweb.cgi
 
 all: perl/Makefile
 	$(MAKE) -C perl
@@ -603,9 +602,6 @@ git$X: git.c common-cmds.h $(BUILTIN_OBJ
 
 help.o: common-cmds.h
 
-git-merge-recur$X: git-merge-recursive$X
-	rm -f $@ && ln git-merge-recursive$X $@
-
 $(BUILT_INS): git$X
 	rm -f $@ && ln git$X $@
 
@@ -918,7 +914,7 @@ check-docs::
 	do \
 		case "$$v" in \
 		git-merge-octopus | git-merge-ours | git-merge-recursive | \
-		git-merge-resolve | git-merge-stupid | git-merge-recur | \
+		git-merge-resolve | git-merge-stupid | \
 		git-merge-recursive-old | \
 		git-ssh-pull | git-ssh-push ) continue ;; \
 		esac ; \

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] Stop making git-merge-recur
  2006-10-02  2:45 [PATCH] Stop making git-merge-recur A Large Angry SCM
@ 2006-10-02  2:49 ` A Large Angry SCM
  2006-10-02  4:41 ` Junio C Hamano
  1 sibling, 0 replies; 4+ messages in thread
From: A Large Angry SCM @ 2006-10-02  2:49 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

A Large Angry SCM wrote:
> Git-merge-recur is no longer installed and make clean no longer
> removes it, so creating making it.
              ^^^^^^^^^^^^^^^^^^
Should be "so stop creating" it.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Stop making git-merge-recur
  2006-10-02  2:45 [PATCH] Stop making git-merge-recur A Large Angry SCM
  2006-10-02  2:49 ` A Large Angry SCM
@ 2006-10-02  4:41 ` Junio C Hamano
  2006-10-02  4:52   ` A Large Angry SCM
  1 sibling, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2006-10-02  4:41 UTC (permalink / raw)
  To: A Large Angry SCM; +Cc: git

A Large Angry SCM <gitzilla@gmail.com> writes:

> Git-merge-recur is no longer installed and make clean no longer
> removes it, so creating making it.

Thanks for noticing.

The bug is actually not installing nor cleaning it.  People who
wanted to stay on the bleeding edge could add "-s recur" on the
command line, or have "merge.twohead = recur" in the config;
this is what we actually encouraged people for wider testing.
We do not suddenly break them.  I planned the removal for the
release after 1.4.3.

This should fix it -- could you eyeball it please?

-- >8 --
Makefile: install and clean merge-recur, still.

We advertised git-merge-recur for some time, and we planned to
support it for one release after we made it the 'recursive'.
However we forgot to install it nor have "make clean" clean it.

Signed-off-by: Junio C Hamano <junkio@cox.net>
---

diff --git a/Makefile b/Makefile
index dd33158..401b893 100644
--- a/Makefile
+++ b/Makefile
@@ -215,7 +215,8 @@ BUILT_INS = \
 	$(patsubst builtin-%.o,git-%$X,$(BUILTIN_OBJS))
 
 # what 'all' will build and 'install' will install, in gitexecdir
-ALL_PROGRAMS = $(PROGRAMS) $(SIMPLE_PROGRAMS) $(SCRIPTS)
+ALL_PROGRAMS = $(PROGRAMS) $(SIMPLE_PROGRAMS) $(SCRIPTS) \
+	git-merge-recur$X
 
 # Backward compatibility -- to be removed after 1.0
 PROGRAMS += git-ssh-pull$X git-ssh-push$X
@@ -586,8 +587,7 @@ export prefix TAR INSTALL DESTDIR SHELL_
 
 ### Build rules
 
-all: $(ALL_PROGRAMS) $(BUILT_INS) git$X gitk gitweb/gitweb.cgi \
-	git-merge-recur$X
+all: $(ALL_PROGRAMS) $(BUILT_INS) git$X gitk gitweb/gitweb.cgi
 
 all: perl/Makefile
 	$(MAKE) -C perl

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] Stop making git-merge-recur
  2006-10-02  4:41 ` Junio C Hamano
@ 2006-10-02  4:52   ` A Large Angry SCM
  0 siblings, 0 replies; 4+ messages in thread
From: A Large Angry SCM @ 2006-10-02  4:52 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Junio C Hamano wrote:
> A Large Angry SCM <gitzilla@gmail.com> writes:
> 
>> Git-merge-recur is no longer installed and make clean no longer
>> removes it, so creating making it.
> 
> Thanks for noticing.
> 
> The bug is actually not installing nor cleaning it.  People who
> wanted to stay on the bleeding edge could add "-s recur" on the
> command line, or have "merge.twohead = recur" in the config;
> this is what we actually encouraged people for wider testing.
> We do not suddenly break them.  I planned the removal for the
> release after 1.4.3.
> 
> This should fix it -- could you eyeball it please?

+1

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-10-02  4:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-02  2:45 [PATCH] Stop making git-merge-recur A Large Angry SCM
2006-10-02  2:49 ` A Large Angry SCM
2006-10-02  4:41 ` Junio C Hamano
2006-10-02  4:52   ` A Large Angry SCM

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).