On Fri, 15 Jul 2005 07:56:36 -0700 Tom Rini wrote: > When running depmod to check for the correct version number, extra > output we don't need to see, such as "depmod: QM_MODULES: Function not > implemented" may show up. Redirect stderr to /dev/null as the version > information that we do care about comes to stdout. > > Signed-off-by: Tom Rini <[EMAIL PROTECTED]> > > diff --git a/Makefile b/Makefile > --- a/Makefile > +++ b/Makefile > @@ -875,7 +875,7 @@ modules_install: _modinst_ _modinst_post > > .PHONY: _modinst_ > _modinst_: > - @if [ -z "`$(DEPMOD) -V | grep module-init-tools`" ]; then \ > + @if [ -z "`$(DEPMOD) -V 2>/dev/null | grep module-init-tools`" ]; then \ > echo "Warning: you may need to install module-init-tools"; \ > echo "See > http://www.codemonkey.org.uk/docs/post-halloween-2.6.txt";\ > sleep 1; \
Well, seeing "QM_MODULES" is a great indicator that someone is using modutils instead of module-init-tools, so I'd like to see it stay. IOW, I somewhat disagree with "extra output we don't need to see." --- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
