Re: [PATCH] nmbug-status: print config errors to stderr

[David Bremner]

Jani Nikula  writes:

> Particularly scripted usage with stdout redirection can be confusing
> if errors are printed to stdout instead of stderr.

pushed

d
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH] nmbug-status: print config errors to stderr

[Tomi Ollila]

On Sat, Oct 03 2015, Jani Nikula  wrote:

> Particularly scripted usage with stdout redirection can be confusing
> if errors are printed to stdout instead of stderr.

LGTM

> ---
>  devel/nmbug/nmbug-status | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/devel/nmbug/nmbug-status b/devel/nmbug/nmbug-status
> index a289798e3cc1..b36b6ad33859 100755
> --- a/devel/nmbug/nmbug-status
> +++ b/devel/nmbug/nmbug-status
> @@ -318,7 +318,7 @@ args = parser.parse_args()
>  try:
>  config = read_config(path=args.config)
>  except ConfigError as e:
> -print(e)
> +print(e, file=sys.stderr)
>  sys.exit(1)
>  
>  header_template = config['meta'].get('header', '''
> -- 
> 2.1.4
>
> ___
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH] nmbug-status: print config errors to stderr

[W. Trevor King]

On Sat, Oct 03, 2015 at 12:08:05PM +0300, Tomi Ollila wrote:
> On Sat, Oct 03 2015, Jani Nikula wrote:
> > Particularly scripted usage with stdout redirection can be
> > confusing if errors are printed to stdout instead of stderr.
> 
> LGTM

Me too.

Cheers,
Trevor

-- 
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy


signature.asc
Description: OpenPGP digital signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch