[PATCH 5/5] doc: filter out doc generation spam

Peter Hutterer peter.hutterer at who-t.net
Tue Dec 9 16:07:15 PST 2014


On Tue, Dec 09, 2014 at 11:12:47AM -0800, Keith Packard wrote:
> Julien Cristau <jcristau at debian.org> writes:
> 
> > That means we now ignore the return value from xmlto, and only look at
> > that of grep.  I'm not sure that's what we want.
> 
> How about this instead? This captures the xmlto exit status and routes
> it around the error message filtering.
> 
> diff --git a/devbook.am b/devbook.am
> index 400b2ca..983cc98 100644
> --- a/devbook.am
> +++ b/devbook.am
> @@ -46,9 +46,10 @@ XMLTO_FO_FLAGS =                                     \
>  
>  noinst_DATA += $(docbook:.xml=.pdf) $(docbook:.xml=.ps)
>  %.pdf: %.xml $(chapters)
> -       $(AM_V_GEN)$(XMLTO) $(XMLTO_FO_FLAGS) --with-fop pdf $<
> +       $(AM_V_GEN)(((($(XMLTO) $(XMLTO_FO_FLAGS) --with-fop pdf $< 2>&1; echo $$? >&3) | grep -v 'overflows' | grep -v Rendered >&4) 3>&1) | (read status; exit $$status)) 4>&1
> +
>  %.ps: %.xml $(chapters)
> -       $(AM_V_GEN)$(XMLTO) $(XMLTO_FO_FLAGS) --with-fop ps $<
> +       $(AM_V_GEN)(((($(XMLTO) $(XMLTO_FO_FLAGS) --with-fop ps $< 2>&1; echo $$? >&3) | grep -v 'overflows' | grep -v Rendered >&4) 3>&1) | (read status; exit $$status)) 4>&1
>  endif HAVE_FOP
>  endif HAVE_STYLESHEETS

I wonder if we've reached the point where having a small shell script to build
this with the reduced output is easier than stuffing this into makefile
rules...

Cheers,
   Peter




More information about the xorg-devel mailing list