On 10/05/2015 02:00 AM, Martin Kletzander wrote:
On Mon, Oct 05, 2015 at 09:18:41AM +0200, Michal Privoznik wrote:
On 01.10.2015 14:39, Martin Kletzander wrote:
We are distributing virkeymaps.h and all the tools needed to rebuild that file. On top of that, we are generating that file into the $(srcdir) and that sometimes fails for me when trying to do make dist in VPATH on rawhide fedora. And we don't clean the file when maintainer-clean make target is requested.
Signed-off-by: Martin Kletzander <mkletzan@redhat.com> ---
-EXTRA_DIST += $(srcdir)/util/virkeymaps.h $(srcdir)/util/keymaps.csv \ - $(srcdir)/util/virkeycode-mapgen.py -
I am afraid we can't do this. What if somebody wants to 'make dist' from unpacked .tar.gz? They must have both .csv and .py files. So you can't really drop them from here. What you can drop is .h file.
Oh so that's probably what Daniel meant in the first proposition of this patch. I misunderstood that. OK, well, I think making dist from unpacked dist doesn't make much sense, but why would we block it, right?
In fact, 'make dist-check' does precisely that - it runs 'make dist' from an unpacked dist, to make sure you didn't forget any source files that were required to generate something that is not shipped pre-generated. So I agree that we have to ship the .csv and .py, and only omit the .h. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org