[FFmpeg-devel] [PATCH 5/5] build: rename version.h to libavutil/ffversion.h

[James Almer]

On 28/11/13 11:38 PM, Timothy Gu wrote:
> On Nov 28, 2013 6:25 PM, "James Almer" <jamrial at gmail.com> wrote:
>>
>> On 28/11/13 10:00 PM, Timothy Gu wrote:
>>> Also the libavutil/ffversion.h will be installed.
>>>
>>> Rationale:
>>> * Applications might want to know FFmpeg's version besides the
> individual
>>>   libraries'.
>>> * Avoids file name clash between FFmpeg's ./version.h and
> lib*/version.h when
>>>   a library source file includes both and is compiled on an out-of-tree
> build.
>>>
>>
>> Missing minor bump for lavu.
> 
> Do I need to add it to APIChanges?
> 
It's a new installed header, so I'd say yes.

>> You could add a line here mentioning this closes ticket #1769.
> 
> This doesn't close #1769. The ticket is about configure options, this is
> about version. (I have a separate patch for that too, see the ticket)

I interpreted that ticket as requesting the ffmpeg build string from version.h, 
and not the list of configure arguments, being available outside of the build 
environment.

If you're reading this Don Moir, could you clear things up?

> 
>>
>> Looks good, assuming renaming version.h is accepted.
> 
> [...]
> 
> Timothy
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>