Re: [PATCH v6 31/44] run-command: make dup_devnull() non static
On Sat, Jun 11, 2016 at 10:17 AM, Johannes Sixtwrote: > Am 10.06.2016 um 22:11 schrieb Christian Couder: >> >> --- a/run-command.c >> +++ b/run-command.c >> @@ -85,7 +85,7 @@ static inline void close_pair(int fd[2]) >> } >> >> #ifndef GIT_WINDOWS_NATIVE >> -static inline void dup_devnull(int to) >> +void dup_devnull(int to) > > > I'm not adding arguments to what we've heard on whether to use /dev/null in > this series or not. But if the outcome is to keep this patch, please remove > the #ifndef that we see in the context lines (and the matching #endif), too. > Otherwise, the build fails on Windows for each patch in the series until > this change is reverted in patch 42/44. Ok, I will find a way to avoid that build failure, though I don't test on Windows. Thanks, Christian. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v6 31/44] run-command: make dup_devnull() non static
Am 10.06.2016 um 22:11 schrieb Christian Couder: --- a/run-command.c +++ b/run-command.c @@ -85,7 +85,7 @@ static inline void close_pair(int fd[2]) } #ifndef GIT_WINDOWS_NATIVE -static inline void dup_devnull(int to) +void dup_devnull(int to) I'm not adding arguments to what we've heard on whether to use /dev/null in this series or not. But if the outcome is to keep this patch, please remove the #ifndef that we see in the context lines (and the matching #endif), too. Otherwise, the build fails on Windows for each patch in the series until this change is reverted in patch 42/44. -- Hannes -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v6 31/44] run-command: make dup_devnull() non static
We will need this function in a later commit to redirect stdout and stderr to /dev/null. Helped-by: Johannes SixtHelped-by: Johannes Schindelin Signed-off-by: Christian Couder --- run-command.c | 2 +- run-command.h | 6 ++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/run-command.c b/run-command.c index af0c8a1..d8ed88c 100644 --- a/run-command.c +++ b/run-command.c @@ -85,7 +85,7 @@ static inline void close_pair(int fd[2]) } #ifndef GIT_WINDOWS_NATIVE -static inline void dup_devnull(int to) +void dup_devnull(int to) { int fd = open("/dev/null", O_RDWR); if (fd < 0) diff --git a/run-command.h b/run-command.h index 11f76b0..e05ce7d 100644 --- a/run-command.h +++ b/run-command.h @@ -201,4 +201,10 @@ int run_processes_parallel(int n, task_finished_fn, void *pp_cb); +/** + * Misc helper functions + */ + +void dup_devnull(int to); + #endif -- 2.9.0.rc2.362.g3cd93d0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html