# [PATCH 1/2] daemon: send more error messages to the syslog

2 messages from 2008-02-25 to 2008-02-25. Participants: Johannes Sixt, Junio C Hamano.
Thread: https://gitlist.dev/t/12309

## Johannes Sixt, 2008-02-25 13:25

Subject: [PATCH 1/2] daemon: send more error messages to the syslog
Message-ID: <47C2C1C0.5070903@viscovery.net>
URL: https://gitlist.dev/e/47C2C1C0.5070903%40viscovery.net

```
There were a number of die() calls before the syslog was opened; hence,
these error messages would have been sent to /dev/null in detached mode.
Now we install the daemon-specific die routine before any error message is
generated so that these messages go to the syslog.

Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
---
 daemon.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/daemon.c b/daemon.c
index 41a60af..dd0177f 100644
--- a/daemon.c
+++ b/daemon.c
@@ -1149,6 +1149,11 @@ int main(int argc, char **argv)
 		usage(daemon_usage);
 	}

+	if (log_syslog) {
+		openlog("git-daemon", 0, LOG_DAEMON);
+		set_die_routine(daemon_die);
+	}
+
 	if (inetd_mode && (group_name || user_name))
 		die("--user and --group are incompatible with --inetd");

@@ -1176,11 +1181,6 @@ int main(int argc, char **argv)
 		}
 	}

-	if (log_syslog) {
-		openlog("git-daemon", 0, LOG_DAEMON);
-		set_die_routine(daemon_die);
-	}
-
 	if (strict_paths && (!ok_paths || !*ok_paths))
 		die("option --strict-paths requires a whitelist");

-- 
1.5.4.3.229.g5c72

```

## Junio C Hamano, 2008-02-25 19:02

Subject: Re: [PATCH 1/2] daemon: send more error messages to the syslog
Message-ID: <7v3argkgfe.fsf@gitster.siamese.dyndns.org>
URL: https://gitlist.dev/e/7v3argkgfe.fsf%40gitster.siamese.dyndns.org
In-Reply-To: <47C2C1C0.5070903@viscovery.net>

```
Johannes Sixt <j.sixt@viscovery.net> writes:

> There were a number of die() calls before the syslog was opened; hence,
> these error messages would have been sent to /dev/null in detached mode.
> Now we install the daemon-specific die routine before any error message is
> generated so that these messages go to the syslog.

Makes sense.  Thanks.

```
