[FFmpeg-cvslog] lavfi: Add infrastructure for building OpenCL source into libavfilter

Mark Thompson sw at jkqxz.net
Thu Nov 23 01:57:39 EET 2017


On 22/11/17 23:49, Carl Eugen Hoyos wrote:
> 2017-11-23 0:38 GMT+01:00 Mark Thompson <sw at jkqxz.net>:
>> On 22/11/17 23:28, Carl Eugen Hoyos wrote:
>>> 2017-11-23 0:22 GMT+01:00 Mark Thompson <git at videolan.org>:
>>>> ffmpeg | branch: master | Mark Thompson <sw at jkqxz.net> | Sun Sep 10 21:53:35 2017 +0100| [59d6529333c4550701401d59d67b60454b164b1f] | committer: Mark Thompson
>>>>
>>>> lavfi: Add infrastructure for building OpenCL source into libavfilter
>>>>
>>>>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=59d6529333c4550701401d59d67b60454b164b1f
>>>
>>>> diff --git a/tools/cl2c b/tools/cl2c
>>>> new file mode 100755
>>>> index 0000000000..1ead940811
>>>> --- /dev/null
>>>> +++ b/tools/cl2c
>>>> @@ -0,0 +1,20 @@
>>>> +#!/bin/sh
>>>> +# Convert an OpenCL source file into a C source file containing the
>>>> +# OpenCL source as a C string.  Also adds a #line directive so that
>>>> +# compiler messages are useful.
>>>> +
>>>
>>> Don't we need a license header to make distributions and fate happy?
>>
>> Other short scripts in the tools directory don't have one, I followed that pattern.
>>
>> Fate doesn't mind because only looks at real source files ("grep -E '\.c$|\.h$|\.S$|\.asm$'").
>>
>> I can add one anyway if you think that's sensible?
> 
> Sorry, I just thought that Debian (and friends?) need one.
> Since this is apparently wrong, it's (only) your decision.

We spent more time thinking about it than adding one would take :P

It doesn't matter to me; I've added one in case it matters to someone else.

Thanks,

- Mark


More information about the ffmpeg-cvslog mailing list