Eric Blake wrote: > According to Jim Meyering on 2/24/2010 6:20 AM: >> Running "make distcheck" from a non-srcdir build was failing like this: >> >> /bin/sh: ./apibuild.py: No such file or directory >> >> Fixed that with the first patch below (the apibuild.py script >> is not in ".", but in $(srcdir)). >> >> Then retrying, it got a little further but failed like this: >> >> (../../docs/apibuild.py) >> make[3]: *** No rule to make target `sitemap.html.in', needed by `html/index.html'. Stop. >> >> That was because sitemap.html.in was not being distributed. >> Adding it to EXTRA_DIST solved that: > > ACK to both patches, but a question... Thanks for the reviews. >> @@ -131,7 +131,7 @@ devhelp/index.html devhelp/libvirt.devhelp: libvirt-api.xml $(devhelpxsl) >> libvirt-api.xml libvirt-refs.xml: apibuild.py \ >> $(srcdir)/../include/libvirt/*.h \ >> $(srcdir)/../src/libvirt.c $(srcdir)/../src/util/virterror.c >> - -(./apibuild.py) >> + -($(srcdir)/apibuild.py) > > Why the subshell? I wondered the same thing. I can see no way it would matter, so I'll remove it as part of that patch: >From 6d5aebcc2eb120e34d782c0740e607aecfb1900c Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering@xxxxxxxxxx> Date: Wed, 24 Feb 2010 14:18:40 +0100 Subject: [PATCH] build: avoid non-srcdir installation failure (apibuild.py) * docs/Makefile.am (libvirt-api.xml): Insert missing "$(srcdir)/". Also, remove unnecessary sub-shell. --- docs/Makefile.am | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/docs/Makefile.am b/docs/Makefile.am index cce7a48..9065ef1 100644 --- a/docs/Makefile.am +++ b/docs/Makefile.am @@ -131,7 +131,7 @@ devhelp/index.html devhelp/libvirt.devhelp: libvirt-api.xml $(devhelpxsl) libvirt-api.xml libvirt-refs.xml: apibuild.py \ $(srcdir)/../include/libvirt/*.h \ $(srcdir)/../src/libvirt.c $(srcdir)/../src/util/virterror.c - -(./apibuild.py) + -$(srcdir)/apibuild.py clean-local: rm -f *~ *.bak *.hierarchy *.signals *-unused.txt -- 1.7.0.377.g90adb -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list