[PATCH] run-command.c: remove run_command_v_opt_cd()

Subsystems: the rest

DORMANTno replies

3 messages, 3 authors, 2016-06-15 · open the first message on its own page

[PATCH] run-command.c: remove run_command_v_opt_cd()

From: Nanako Shiraishi <hidden>
Date: 2016-06-15 22:45:26

This function is not used anywhere.

Signed-off-by: Nanako Shiraishi <redacted>
---
 run-command.c |    8 --------
 run-command.h |    1 -
 2 files changed, 0 insertions(+), 9 deletions(-)
diff --git a/run-command.c b/run-command.c
index caab374..c90cdc5 100644
--- a/run-command.c
+++ b/run-command.c
@@ -273,14 +273,6 @@ int run_command_v_opt(const char **argv, int opt)
 	return run_command(&cmd);
 }
 
-int run_command_v_opt_cd(const char **argv, int opt, const char *dir)
-{
-	struct child_process cmd;
-	prepare_run_command_v_opt(&cmd, argv, opt);
-	cmd.dir = dir;
-	return run_command(&cmd);
-}
-
 int run_command_v_opt_cd_env(const char **argv, int opt, const char *dir, const char *const *env)
 {
 	struct child_process cmd;
diff --git a/run-command.h b/run-command.h
index 4f2b7d7..a8b0c20 100644
--- a/run-command.h
+++ b/run-command.h
@@ -53,7 +53,6 @@ int run_command(struct child_process *);
 #define RUN_GIT_CMD	     2	/*If this is to be git sub-command */
 #define RUN_COMMAND_STDOUT_TO_STDERR 4
 int run_command_v_opt(const char **argv, int opt);
-int run_command_v_opt_cd(const char **argv, int opt, const char *dir);
 
 /*
  * env (the environment) is to be formatted like environ: "VAR=VALUE".
-- 
1.6.0.2

-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/

Re: [PATCH] run-command.c: remove run_command_v_opt_cd()

From: Johannes Sixt <hidden>
Date: 2016-06-15 22:45:26

Nanako Shiraishi schrieb:
This function is not used anywhere.
Indeed. Future callers can use run_command_v_opt_cd_env() instead.

The function is also mentioned in
Documentation/technical/api-run-command.txt, where you should remove it, too.

-- Hannes

Re: [PATCH] run-command.c: remove run_command_v_opt_cd()

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:45:26

Hi,

On Thu, 2 Oct 2008, Nanako Shiraishi wrote:
This function is not used anywhere.
I have to say that it fills me with sorrow, seeing that the direction 
these patches take is the _opposite_ of libifying Git.  So far I was only 
disappointed that nobody pushed this goal, but now I am concerned that 
anybody thinking about it is actively discouraged.

Just think about Git Cheetah: if it were ever to benefit from an official 
libgit, then the function you remove would be _ideal_.  By removing that 
function, you make it even less attractive to try to use Git as a proper 
library.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help