[libvirt] [PATCH] build: Use $(PYTHON) instead of python for the keycode map generator

Daniel Veillard veillard at redhat.com
Fri Jul 22 14:17:32 UTC 2011


On Fri, Jul 22, 2011 at 08:06:00AM -0600, Eric Blake wrote:
> On 07/22/2011 07:56 AM, Matthias Bolte wrote:
> >Also prepend $(AM_V_GEN).
> >---
> >  src/Makefile.am |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> >diff --git a/src/Makefile.am b/src/Makefile.am
> >index 481caba..eef0669 100644
> >--- a/src/Makefile.am
> >+++ b/src/Makefile.am
> >@@ -90,7 +90,7 @@ EXTRA_DIST += $(srcdir)/util/virkeymaps.h $(srcdir)/util/keymaps.csv \
> >
> >  $(srcdir)/util/virkeymaps.h: $(srcdir)/util/keymaps.csv	\
> >  		$(srcdir)/util/virkeycode-mapgen.py
> >-	python $(srcdir)/util/virkeycode-mapgen.py<$(srcdir)/util/keymaps.csv>$@
> >+	$(AM_V_GEN)$(PYTHON) $(srcdir)/util/virkeycode-mapgen.py<$(srcdir)/util/keymaps.csv>$@
> 
> ACK.

  Ah right, I missed that !

> Hmm, while you're at it, how about you also:
> 
> chmod +x src/util/virkeycode-mapgen.py
> 
> to match our style of .py files being executable (even if we run
> them via $(PYTHON)).

  just wondering loud, if you change a file attribute like this will
git automatically detect it and all to "commit" the change or is there
some magic involved (yes I'm think thinking too much in terms of CVS !)

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list