On Mon, 2009-12-14 at 10:41 +0200, Vasilis Tsiligiannis wrote: 
> Hello Brian,

Hi Vasilis,

> > -log { source(src); destination(messages); };
> > +log { source(src); source(kernel); destination(syslog_server); };
> 
> Correct me if I am wrong: shouldn't there be a "destination syslog_server 
> {...};" also present in the config? Or just send them to messages destination 
> which already exists?

Doh!  More than I wanted leaked into that patch.  I do use a syslog
server here, but I am not convinced that logging to a syslog server was
a reasonable default so didn't include the "destination syslog_server".
But of course I forgot to change the "destination(syslog_server)" back
to "destination(messages)".  Find below a new, cleaner patch.

P.S. what really needs to happen is for the syslog-ng.conf file to be
uci-ized (i.e. generated a run time based on UCI configuration).  I
don't really have time for that at the moment though.  :-(

Signed-off-by: Brian J. Murrell <[email protected]>

---

Index: files/syslog-ng.conf
===================================================================
--- files/syslog-ng.conf        (revision 18617)
+++ files/syslog-ng.conf        (working copy)
@@ -10,7 +10,8 @@
 };
 
 source src { unix-stream("/dev/log"); internal(); };
+source kernel { pipe("/proc/kmsg" log_prefix("kernel: ")); };
 
 destination messages { file("/var/log/messages"); };
 
-log { source(src); destination(messages); };
+log { source(src); source(kernel); destination(messages); };


Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to