[brussels-dev] Brussels- nddcompat code review
Garrett D'Amore
gdamore at opensolaris.org
Fri Mar 28 17:10:49 PDT 2008
Peter Memishian wrote:
> > http://zhadum.east/export/ws/sowmini/brussels/nddcompatchild/webrev
> > where I tried playing with this. But this is going to break any script
> > that parses the output of ndd (I've fprintf'ed to stderr, but I'm not
> > sure that covers every case).
>
> Garrett's suggestion of using isatty() seems fine, but more generally, any
> script that's parsing /dev/stderr is way off in the weeds.
>
Yeah, although I've seen things that tie stderr and stdout together....
putting warnings on stderr is probably not terribly useful anyway ...
(One could drop a message to syslog if isatty() failed, but you'd not
want to do it too often (no spamming of syslog, please), which would
require some kind of state keeping... may not be worth the effort.)
-- Garrett
More information about the brussels-dev
mailing list