X-Original-To: alpine-aports@lists.alpinelinux.org Delivered-To: alpine-aports@mail.alpinelinux.org Received: from [172.17.48.69] (unknown [74.117.191.11]) (using TLSv1 with cipher ECDHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: nangel@tetrasec.net) by mail.alpinelinux.org (Postfix) with ESMTPSA id 78865DC0096 for ; Thu, 12 Mar 2015 13:45:18 +0000 (UTC) Message-ID: <5501986D.9070902@nothome.org> Date: Thu, 12 Mar 2015 09:45:17 -0400 From: Nathan Angelacos User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.4.0 X-Mailinglist: alpine-aports Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 To: alpine-aports@lists.alpinelinux.org Subject: Re: [alpine-aports] [PATCH] main/sysklogd: remove GNUism and use POSIX Extended Regular Expressions (EREs) References: <1426109305-19657-1-git-send-email-nangel@alpinelinux.org> <20150312081159.5e532b9b@ncopa-desktop.alpinelinux.org> <201503121305.17253.vkrishn4@gmail.com> In-Reply-To: <201503121305.17253.vkrishn4@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit >> >> I think you need to change the last line to: >> >> -e "/^.*(auth)[^ ]* /${skip}D" \ >> Thanks Natanael, you are right. Looks like you went ahead and fixed my patch. Thanks! >> Test case: >> >> $ echo "aaabbb" | sed -E -e 's/\(a+\)\(b+\)/\2\1/' >> aaabbb >> $ echo "aaabbb" | sed -E -e 's/(a+)(b+)/\2\1/' >> bbbaaa > > Could someone also run this test after "apk add sed" > and see if they are as expected ? tl;dr - it works as expected with both busybox and gnu sed. Sysklogd.daily, just printing out /etc/syslog.conf: # ./sysklogd.daily /var/log/cron.log /var/log/daemon.log /var/log/kern.log /var/log/lpr.log /var/log/mail.log /var/log/user.log And now with --auth /var/log/auth.log /var/log/debug /var/log/messages /var/log/syslog # apk add sed (1/1) Installing sed (4.2.2-r0) Executing busybox-1.23.1-r0.trigger OK: 49 MiB in 41 packages # ./sysklogd /var/log/cron.log /var/log/daemon.log /var/log/kern.log /var/log/lpr.log /var/log/mail.log /var/log/user.log And now with --auth /var/log/auth.log /var/log/debug /var/log/messages /var/log/syslog ----- # apk del sed (1/1) Purging sed (4.2.2-r0) Executing sed-4.2.2-r0.post-deinstall Executing busybox-1.23.1-r0.trigger # echo "aaabbb" | sed -E -e 's/\(a+\)\(b+\)/\2\1/' aaabbb # echo "aaabbb" | sed -E -e 's/(a+)(b+)/\2\1/' bbbaaa # apk add sed (1/1) Installing sed (4.2.2-r0) Executing busybox-1.23.1-r0.trigger OK: 49 MiB in 41 packages # echo "aaabbb" | sed -E -e 's/\(a+\)\(b+\)/\2\1/' sed: -e expression #1, char 20: invalid reference \2 on `s' command's RHS # echo "aaabbb" | sed -E -e 's/(a+)(b+)/\2\1/' bbbaaa > > --- Unsubscribe: alpine-aports+unsubscribe@lists.alpinelinux.org Help: alpine-aports+help@lists.alpinelinux.org ---