On 28 October 2016 at 19:18, Lionel Landwerlin
<[email protected]> wrote:
> On 28/10/16 19:12, Emil Velikov wrote:
>>
>> From: Emil Velikov <[email protected]>
>>
>> There is nothing in here that would require it.
>>
>> Cc: [email protected]
>> Cc: Lionel Landwerlin <[email protected]>
>> Signed-off-by: Emil Velikov <[email protected]>
>> ---
>> 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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
