[master] 9e754cc Polish verbose compliance in varnish.m4

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Mon Apr 24 17:32:06 CEST 2017


commit 9e754cc5dcd6b6461ce97af5b2ee52fec5161621
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Mon Apr 24 16:38:52 2017 +0200

    Polish verbose compliance in varnish.m4
    
    This is a cosmetic tweak, it doesn't change any behavior. No need to
    bump the serial for that. The benefits are a better output of silent
    make executions, and the preservation of configured variables like
    $(PYTHON) all the way downstream.
    
    A word on this stunt:
    
        \$(A""M_V_GEN)
    
    The backslash is here to escape the dollar sign, nothing too surprising.
    However the expression between the brackets is rejected by autoconf
    because it looks like an undefined macro. To prevent that, the string is
    closed and reopen at once to break the token-inside-brackets construct.

diff --git a/varnish.m4 b/varnish.m4
index 400e087..82514ea 100644
--- a/varnish.m4
+++ b/varnish.m4
@@ -205,10 +205,10 @@ vmod_$1.lo: vcc_$1_if.c vcc_$1_if.h
 vcc_$1_if.h vmod_$1.rst vmod_$1.man.rst: vcc_$1_if.c
 
 vcc_$1_if.c: vmod_$1.vcc
-	\$(AM_V_VMODTOOL) $PYTHON $VMODTOOL -o vcc_$1_if \$(srcdir)/vmod_$1.vcc
+	\$(AM_V_VMODTOOL) \$(PYTHON) \$(VMODTOOL) -o vcc_$1_if \$(srcdir)/vmod_$1.vcc
 
 vmod_$1.3: vmod_$1.man.rst
-	$RST2MAN vmod_$1.man.rst vmod_$1.3
+	\$(A""M_V_GEN) \$(RST2MAN) vmod_$1.man.rst vmod_$1.3
 
 clean: clean-vmod-$1
 



More information about the varnish-commit mailing list