Date: Fri, 20 Sep 2002 08:37:42 +1000 (EST) From: Bruce Evans <bde@zeta.org.au> To: Mark Murray <mark@grondar.za> Cc: arch@FreeBSD.ORG Subject: Re: More lint work for share/mk/ (review, please) Message-ID: <20020920083332.E3384-100000@gamplex.bde.org> In-Reply-To: <200209192115.g8JLF57s003495@grimreaper.grondar.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 19 Sep 2002, Mark Murray wrote: > > % - ${LINT} ${LINTFLAGS} ${CFLAGS:M-[DIU]*} ${.ALLSRC} | more 2>&1 > > % + ${LINT} ${LINTFLAGS} ${CFLAGS:M-[DIU]*} ${.ALLSRC} > > > > I wonder what this pipeline is for. I once thought that it was to combine > > stdout with stderr to work around the bug that lint prints some messages > > on stderr, but it is wrong for that -- the correct way to combine stdin > > with stderr is "2>&1" before any pipe. > > This got discussed to death last time. It is minor and mostly irrelevant; > its here for consistency with another similar removal. Locally I just > put it back in the .mk file as I disagreed with that removal too, and put it back in some places (mainly as a reminder to fix it properly). > lint: > ${LINT} .... ${.ALLSRC} 2>&1 | cat > > so I can > > # make lint | less > > I don't feel stronly enough to discuss this any further. Take it > out, leave it in. Either way suits me. :-) The "| cat" part brings out my anti-useless process reflex :-). Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020920083332.E3384-100000>