[FFmpeg-devel] [PATCH] Implement postproc_version()

[Ramiro Polla]

On Mon, Sep 1, 2008 at 7:14 AM, M?ns Rullg?rd <mans at mansr.com> wrote:
>
> Stefano Sabatini wrote:
>> On date Monday 2008-09-01 09:59:34 +0200, Diego Biurrun encoded:
>>> On Mon, Sep 01, 2008 at 12:03:33AM +0200, Stefano Sabatini wrote:
>>> >
>>> > BTW, I experienced that is currently quite hard to understand simply
>>> > reading the log which are the versions where some feature was
>>> > introduced, this usually require a painful search through the cvslog
>>> > archive.
>>> >
>>> > I propose so to implement an SVN hook to add automatically to the log
>>> > the version number for each library updated by a commit.
>>>
>>> I don't understand what you are talking about, please try to explain
>>> again.
>>
>> What I mean is something which would add to the committed log the
>> version of the libraries whose number has been updated.
>> For example:
>>
>> |Implement postproc_version().
>> |
>> |libpostproc updated version: 51.2.0
>>  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> This would be added by the SVN hook.
>
> Version changes are already in the diff.

And so are the names of the files that have changed. And if you
understand a certain group of file names to be a module, there is no
need to put the module name in the first line of the log message.

I really don't understand your logic, Mans. One day you're going
berserk because someone didn't explicitely mention in the log what
part of FFmpeg they've changed, the next you say it's "already in the
diff".