On 28 October 2016 at 19:18, Lionel Landwerlin
<lionel.g.landwer...@intel.com> wrote:
> On 28/10/16 19:12, Emil Velikov wrote:
>>
>> From: Emil Velikov <emil.veli...@collabora.com>
>>
>> There is nothing in here that would require it.
>>
>> Cc: mesa-sta...@lists.freedesktop.org
>> Cc: Lionel Landwerlin <lionel.g.landwer...@intel.com>
>> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
>> ---
>> Not sure why we got that dependency in the first place, Lionel ?
>
>
> Some of the Makefile's content ends up in the header (the last echo line of
> this patch).
> If someone changes that, it's nice not to have to cleanup all the intel
> directory
>
Valid point. At the same time, this applies to every rule [we have in
mesa] and we don't bother anywhere else.

I'm kind of split, bth.

Mildly related: does the following read too cryptic ? It achieves the
exact same purpose the current code :-)

+       $(AM_V_GEN) $(XXD) -i $< >> $@; \
+       $(SED) -i -e '1 s,^.*$$,static const uint8_t $(*F)_xml[] = {,;$$ d' $@

Thanks
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to