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(a)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