Re: [PATCH] examples: Fix syslog LOGSCRIPT

2015-03-03 Thread Laurent Bercot

On 03/03/2015 23:34, Olivier Brunel wrote:

Log lines are actually prefixed with uids from $IPCREMOTEEUID & $IPCREMOTEEGID,
so they should be acocunted for in the regexs.


 Ah, thanks. Will apply it.



Also note the need to use \s because, AFAIK, there's no way to use spaces in the
regex then, as space is a delimiter for splitting. This is probably important
enough to be noted alongside in some README in fact, since that's why I've
personally kept them in the run file.


 Hm. A quick solution would be to only split on \t, \n and \r, and use tabs
instead of spaces in env/LOGSCRIPT. But that's ugly.

--
 Laurent



Re: [PATCH] examples: Fix syslog LOGSCRIPT

2015-03-03 Thread Colin Booth
On Mar 3, 2015 2:35 PM, "Olivier Brunel"  wrote:
>
> Log lines are actually prefixed with uids from $IPCREMOTEEUID &
$IPCREMOTEEGID,
> so they should be acocunted for in the regexs.
>
Damn, you're right.

> Also note the need to use \s because, AFAIK, there's no way to use spaces
in the
> regex then, as space is a delimiter for splitting. This is probably
important
> enough to be noted alongside in some README in fact, since that's why I've
> personally kept them in the run file.
>
I need to double-check the word splitting but there might be a way around
that while keeping the logger args separate (and readable) from the run
script.

Cheers!


[PATCH] examples: Fix syslog LOGSCRIPT

2015-03-03 Thread Olivier Brunel
Log lines are actually prefixed with uids from $IPCREMOTEEUID & $IPCREMOTEEGID,
so they should be acocunted for in the regexs.

Signed-off-by: Olivier Brunel 
---
Also note the need to use \s because, AFAIK, there's no way to use spaces in the
regex then, as space is a delimiter for splitting. This is probably important
enough to be noted alongside in some README in fact, since that's why I've
personally kept them in the run file.

 .../ROOT/img/services-local/syslogd-linux/log/env/LOGSCRIPT| 10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/examples/ROOT/img/services-local/syslogd-linux/log/env/LOGSCRIPT 
b/examples/ROOT/img/services-local/syslogd-linux/log/env/LOGSCRIPT
index 65c53e6..e7a1eb7 100644
--- a/examples/ROOT/img/services-local/syslogd-linux/log/env/LOGSCRIPT
+++ b/examples/ROOT/img/services-local/syslogd-linux/log/env/LOGSCRIPT
@@ -1,6 +1,6 @@
-- +^error\. t /var/log/syslogd/error 
-- +^authpriv\. t /var/log/syslogd/auth 
-- +^user\. t /var/log/syslogd/user 
-- +^messages\. t /var/log/syslogd/messages 
-- +^daemon\. t /var/log/syslogd/daemon 
+- +^[0-9]+:\s[0-9]+:\serror\. t /var/log/syslogd/error
+- +^[0-9]+:\s[0-9]+:\sauthpriv\. t /var/log/syslogd/auth
+- +^[0-9]+:\s[0-9]+:\suser\. t /var/log/syslogd/user
+- +^[0-9]+:\s[0-9]+:\smessages\. t /var/log/syslogd/messages
+- +^[0-9]+:\s[0-9]+:\sdaemon\. t /var/log/syslogd/daemon
 f t /var/log/syslogd/misc
-- 
2.3.1



[announce] execline-2.1.1.0, s6-portable-utils-2.0.3.0

2015-03-03 Thread Laurent Bercot


 Hello,

 * execline-2.1.1.0 is out.
 It adds a new command: forstdin, which splits its standard input and
spawns a program for every element.
 The forbacktickx command is now a wrapper around pipeline and forstdin.

 http://skarnet.org/software/execline/
 git://git.skarnet.org/execline


 * s6-portable-utils-2.0.3.0 is out.
 It adds a new command: s6-dumpenv, which dumps its whole environment
into an envdir.

 http://skarnet.org/software/s6-portable-utils/
 git://git.skarnet.org/s6-portable-utils


 Enjoy,
 Bug-reports welcome.

--
 Laurent