Re: [PATCH 1/2] Makefile: Check availability of `a2ping` and `inkscape`

["Paul E. McKenney" <paulmck@linux.vnet.ibm.com>]

On Tue, Aug 16, 2016 at 10:16:51AM +0900, SeongJae Park wrote:
> This commit modifies Makefile to check availability of `a2ping` and
> `inkscape` command as it already does for `dot` and `fig2eps`.
> 
> Signed-off-by: SeongJae Park <sj38.park@gmail.com>

Applied both, thank you!

							Thanx, Paul

> ---
>  Makefile | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/Makefile b/Makefile
> index b3d7c4c..60e8e96 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -41,6 +41,10 @@ DOT := $(shell which dot 2>/dev/null)
> 
>  FIG2EPS := $(shell which fig2eps 2>/dev/null)
> 
> +A2PING := $(shell which a2ping 2>/dev/null)
> +
> +INKSCAPE := $(shell which inkscape 2>/dev/null)
> +
>  default = $(PERFBOOK_DEFAULT)
> 
>  ifeq ($(default),)
> @@ -127,10 +131,16 @@ endif
> 
>  $(PDFTARGETS_OF_EPS): %.pdf: %.eps
>  	@echo "$< --> $@"
> +ifndef A2PING
> +	$(error "$< --> $@: a2ping not found. Please install it.")
> +endif
>  	@a2ping --below --hires --bboxfrom=compute-gs $< $@ > /dev/null 2>&1
> 
>  $(PDFTARGETS_OF_SVG): %.pdf: %.svg
>  	@echo "$< --> $@"
> +ifndef INKSCAPE
> +	$(error "$< --> $@: inkscape not found. Please install it.")
> +endif
>  	@inkscape --export-pdf=$@ $<
> 
>  clean:
> -- 
> 1.9.1
>