Dnia 2014-10-25, o godz. 12:42:14
Zac Medico <zmed...@gentoo.org> napisał(a):

> On 10/25/2014 09:15 AM, Michael Palimaka wrote:
> > +eqalog() {
> > +   local tag=$1 x
> > +   shift
> > +   for x in "$@" ; do
> > +           echo "${tag}" "${x}" >> "${T}"/qa.log
> > +   done
> > +}
> 
> Your patches look good to me, except that I think eqalog should escape
> any \n characters that might be embedded in the arguments.

Why? That sounds like some unsafe fancy feature only one person would
use. Embedding newlines in bash is trivial, why make them implicit with
a lot of added complexity?

-- 
Best regards,
Michał Górny

Attachment: signature.asc
Description: PGP signature

Reply via email to