[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[libvirt] [PATCH 2/4] util: Remove unused virExec wrapper



v3:
    Remove obsolete comment

Signed-off-by: Cole Robinson <crobinso redhat com>
---
 cfg.mk                   |    1 -
 src/libvirt_private.syms |    1 -
 src/util/util.c          |   35 -----------------------------------
 src/util/util.h          |    8 --------
 4 files changed, 0 insertions(+), 45 deletions(-)

diff --git a/cfg.mk b/cfg.mk
index 3a10186..d16d8a8 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -263,7 +263,6 @@ sc_prohibit_close:
 	  $(_sc_search_regexp)
 
 # Prefer virCommand for all child processes.
-# XXX - eventually, we want to enhance this to also prohibit virExec.
 sc_prohibit_fork_wrappers:
 	@prohibit='= *\<(fork|popen|system) *\('			\
 	halt='use virCommand for child processes'			\
diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 7b6151c..fc6eeaf 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -924,7 +924,6 @@ virEnumFromString;
 virEnumToString;
 virEventAddHandle;
 virEventRemoveHandle;
-virExec;
 virExecWithHook;
 virFileAbsPath;
 virFileDeletePid;
diff --git a/src/util/util.c b/src/util/util.c
index 89fc82b..7679834 100644
--- a/src/util/util.c
+++ b/src/util/util.c
@@ -752,26 +752,6 @@ virExecWithHook(const char *const*argv,
     return -1;
 }
 
-/*
- * See virExecWithHook for explanation of the arguments.
- *
- * Wrapper function for virExecWithHook, with a simpler set of parameters.
- * Used to insulate the numerous callers from changes to virExecWithHook
- * argument list.
- */
-int
-virExec(const char *const*argv,
-        const char *const*envp,
-        const fd_set *keepfd,
-        pid_t *retpid,
-        int infd, int *outfd, int *errfd,
-        int flags)
-{
-    return virExecWithHook(argv, envp, keepfd, retpid,
-                           infd, outfd, errfd,
-                           flags, NULL, NULL, NULL);
-}
-
 /**
  * @argv NULL terminated argv to run
  * @status optional variable to return exit status in
@@ -881,21 +861,6 @@ virRun(const char *const *argv ATTRIBUTE_UNUSED,
 }
 
 int
-virExec(const char *const*argv ATTRIBUTE_UNUSED,
-        const char *const*envp ATTRIBUTE_UNUSED,
-        const fd_set *keepfd ATTRIBUTE_UNUSED,
-        int *retpid ATTRIBUTE_UNUSED,
-        int infd ATTRIBUTE_UNUSED,
-        int *outfd ATTRIBUTE_UNUSED,
-        int *errfd ATTRIBUTE_UNUSED,
-        int flags ATTRIBUTE_UNUSED)
-{
-    virUtilError(VIR_ERR_INTERNAL_ERROR,
-                 "%s", _("virExec is not implemented for WIN32"));
-    return -1;
-}
-
-int
 virExecWithHook(const char *const*argv ATTRIBUTE_UNUSED,
                 const char *const*envp ATTRIBUTE_UNUSED,
                 const fd_set *keepfd ATTRIBUTE_UNUSED,
diff --git a/src/util/util.h b/src/util/util.h
index e2b8eb3..3e95cae 100644
--- a/src/util/util.h
+++ b/src/util/util.h
@@ -69,14 +69,6 @@ int virExecWithHook(const char *const*argv,
                     virExecHook hook,
                     void *data,
                     char *pidfile) ATTRIBUTE_RETURN_CHECK;
-int virExec(const char *const*argv,
-            const char *const*envp,
-            const fd_set *keepfd,
-            pid_t *retpid,
-            int infd,
-            int *outfd,
-            int *errfd,
-            int flags) ATTRIBUTE_RETURN_CHECK;
 int virRun(const char *const*argv, int *status) ATTRIBUTE_RETURN_CHECK;
 int virPipeReadUntilEOF(int outfd, int errfd,
                         char **outbuf, char **errbuf);
-- 
1.7.4.4


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]