From 15fb518cc2bc38a510b54003b59e0152c28c0eec Mon Sep 17 00:00:00 2001 From: "http://www.cse.unsw.edu.au/~willu/" Date: Fri, 18 Jul 2008 21:03:55 -0400 Subject: [PATCH] New bug and patch - close stderr when forking a sub process to run git --- doc/bugs/git_stderr_output_causes_problems.mdwn | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) create mode 100644 doc/bugs/git_stderr_output_causes_problems.mdwn diff --git a/doc/bugs/git_stderr_output_causes_problems.mdwn b/doc/bugs/git_stderr_output_causes_problems.mdwn new file mode 100644 index 000000000..155808884 --- /dev/null +++ b/doc/bugs/git_stderr_output_causes_problems.mdwn @@ -0,0 +1,16 @@ +I've just been getting ikiwiki running on a hosted server. The server is wrapping all cgi scripts to 'harden' them. Unfortunately, that script is sensitive to what a cgi script outputs on stderr. + +Ikiwiki's git handling is sending a bunch of output to stderr. The following patch closes stderr in the child process that ikiwiki forks to run git. This allows me to use ikiwiki on this hosted server. (patch inline - check source to get it) + +diff --git a/IkiWiki/Rcs/git.pm b/IkiWiki/Rcs/git.pm +index 425536f..5734bb2 100644 +--- a/IkiWiki/Rcs/git.pm ++++ b/IkiWiki/Rcs/git.pm +@@ -24,6 +24,7 @@ sub _safe_git (&@) { #{{{ + if (!$pid) { + # In child. + # Git commands want to be in wc. ++ open STDERR, '>/dev/null'; + chdir $config{srcdir} + or error("Cannot chdir to $config{srcdir}: $!"); + exec @cmdline or error("Cannot exec '@cmdline': $!"); -- 2.44.0