[FFmpeg-devel] [PATCH] news: add entry about stricter configure behavior

James Almer jamrial at gmail.com
Sun Dec 11 00:59:03 EET 2016


On 12/10/2016 7:27 PM, Carl Eugen Hoyos wrote:
> 2016-12-10 23:24 GMT+01:00 James Almer <jamrial at gmail.com>:
>> On 12/10/2016 7:22 PM, Carl Eugen Hoyos wrote:
>>> 2016-12-10 19:47 GMT+01:00 Andreas Cadhalpun <andreas.cadhalpun at googlemail.com>:
>>>> Suggested-by: Carl Eugen Hoyos <ceffmpeg at gmail.com>
>>>> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
>>>> ---
>>>>  src/index | 7 +++++++
>>>>  1 file changed, 7 insertions(+)
>>>>
>>>> diff --git a/src/index b/src/index
>>>> index c203676..8f533f5 100644
>>>> --- a/src/index
>>>> +++ b/src/index
>>>> @@ -37,6 +37,13 @@
>>>>      News
>>>>    </h1>
>>>>
>>>> +  <h3 id="stricterconfigure">December 10th, 2016, stricter configure behavior.</h3>
>>>> +  <p>
>>>> +    The configure script now fails if autodetect-libraries are requested but not found.
>>>> +  </p>
>>>> +  <p>
>>>> +    If for example SDL headers are not available, '--enable-sdl2' will cause a configure failure.
>>>
>>> Please push, esp. since the relevant change is already in git.
>>>
>>> Thank you, Carl Eugen
>>
>> Again, I'm against this. This is not news entry worthy.
> 
> Then please revert the configure change:
> This is (by far) the biggest change in configure behaviour since forever,
> this has to be communicated and this was the condition for the patch.

I don't think reverting an (allegedly wanted and beneficial) change just
because you wont accept it unless it has its own news entry makes sense
or is even fair to the person that wrote it and submitted it.

> 
>> A line can be added to the next release news entry announcement, or
>> to the RELEASE_NOTES files, but not alone.
> 
> If you care, add it to the release notes, but this cannot replace the
> news entry.

The RELEASE_NOTES mention is the only thing that's a must for this,
aside from the Changelog entry.



More information about the ffmpeg-devel mailing list