* [PATCH] Makefile: drop extra dependencies for test helpers
@ 2016-07-01 7:56 Jeff King
2016-07-01 7:59 ` [PATCH] Makefile: use VCSSVN_LIB to refer to svn library Jeff King
0 siblings, 1 reply; 2+ messages in thread
From: Jeff King @ 2016-07-01 7:56 UTC (permalink / raw
To: git
A few test-helpers have Makefile dependencies on specific
object files. But since these files are part of libgit.a
(which all of the helpers link against), the inclusion is
simply redundant.
These were once necessary, but became redundant due to
5c5ba73 (Makefile: Use generic rule to build test programs,
2007-05-31), which added the $(GITLIBS) dependency (but
didn't prune the extra dependency lines). Later commits then
cargo-culted the practice (e.g., b4285c7).
Note that we _do_ need to leave the dependencies on the svn
library, as that is not part of the usual link command.
Signed-off-by: Jeff King <peff@peff.net>
---
Just a cleanup I noticed while working on the common-main series.
Makefile | 8 --------
1 file changed, 8 deletions(-)
diff --git a/Makefile b/Makefile
index de5a030..22815a2 100644
--- a/Makefile
+++ b/Makefile
@@ -2225,16 +2225,8 @@ perf: all
.PHONY: test perf
-t/helper/test-ctype$X: ctype.o
-
-t/helper/test-date$X: date.o ctype.o
-
-t/helper/test-delta$X: diff-delta.o patch-delta.o
-
t/helper/test-line-buffer$X: vcs-svn/lib.a
-t/helper/test-parse-options$X: parse-options.o parse-options-cb.o
-
t/helper/test-svn-fe$X: vcs-svn/lib.a
.PRECIOUS: $(TEST_OBJS)
--
2.9.0.317.g65b4e7c
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] Makefile: use VCSSVN_LIB to refer to svn library
2016-07-01 7:56 [PATCH] Makefile: drop extra dependencies for test helpers Jeff King
@ 2016-07-01 7:59 ` Jeff King
0 siblings, 0 replies; 2+ messages in thread
From: Jeff King @ 2016-07-01 7:59 UTC (permalink / raw
To: git
We have an abstracted variable; let's use it consistently.
Signed-off-by: Jeff King <peff@peff.net>
---
On top of the cleanup earlier in the thread.
I actually wonder if we should drop the vcs-svn code entirely. The last
update that wasn't just part of a "I'm grepping the whole code base to
change this interface" patch was almost 4 years ago, and AFAICT it never
reached a shippable point (we _do_ actually install git-remote-testsvn,
but given the name, I sort of assume nobody is using it).
But I don't want to step on the toes of anybody who actually is using
it, or is planning to work on it more.
Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index 22815a2..4579eab 100644
--- a/Makefile
+++ b/Makefile
@@ -2225,9 +2225,9 @@ perf: all
.PHONY: test perf
-t/helper/test-line-buffer$X: vcs-svn/lib.a
+t/helper/test-line-buffer$X: $(VCSSVN_LIB)
-t/helper/test-svn-fe$X: vcs-svn/lib.a
+t/helper/test-svn-fe$X: $(VCSSVN_LIB)
.PRECIOUS: $(TEST_OBJS)
--
2.9.0.317.g65b4e7c
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-07-01 8:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-01 7:56 [PATCH] Makefile: drop extra dependencies for test helpers Jeff King
2016-07-01 7:59 ` [PATCH] Makefile: use VCSSVN_LIB to refer to svn library Jeff King
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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 read-only IMAP folder(s) and NNTP newsgroup(s).