[FFmpeg-devel] [PATCH] avfilter/vidstab: set bytesPerPixel only for packed formats.

Gyan Doshi gyandoshi at gmail.com
Tue Jan 16 14:22:52 EET 2018



On 1/16/2018 5:19 PM, Michael Niedermayer wrote:
> On Thu, Jan 11, 2018 at 12:15:00PM +0530, Gyan Doshi wrote:
>>
>> On 1/11/2018 6:16 AM, Michael Niedermayer wrote:
>>>>>>
>>>>>> Georg, do you want to maintain vf_vidstabtransform.c ?
>>>>>>
>>>>>> If not, who wants to maintain vf_vidstabtransform.c ?
>>>>>>
>>>>>> If noone then iam not sure who will test or apply this, i think many
>>>>>> dont have the dependancy installed ...
>>>>>
>>>>> I assume the author hasn't gotten back yet. Can anyone else test and apply
>>>>> this?
>>>>
>>>> Anyone?
>>>
>>> If noone else cares about maintaining this ...
>>> then there is one radical but simple solution ...
>>> post a patch that adds you to MAINTAINERs for the file.
>>>
>>> Anyone could object yes, but then that person should at least review and apply
>>> the vf_vidstabtransform.c patch
>>
>> I have no interest or business maintaining the library wrappers.
>>
>> In the commit history for these files, I see Paul Mahol, Clément Bœsch and
>> you as committers among those active. Timothy Gu has signed off as well, but
>> I don't know if he's active.
>>
>> So, someone among the above could take a look?
> 
> Please correct me if i misunderstand but IIUC you basically say
> you wont review patches for vf_vidstabtransform (that is what maintaining is)
> but you want other people to review your patches for vf_vidstabtransform

I won't review patches for vidstab because I can't. There is an open 
ticket which has a simple fix which I noticed, applied & tested. But I'm 
not well-versed in the internals of vidstab to be a competent maintainer 
for all other patches, as they may come up.

I believe you suggested to propose myself for maintainership *as a ruse* 
to get someone to review my patch and not in earnest. If you were, it 
would be insincere for me to go through with it.

I can vouch that the patch works at my end, and if that's not enough, it 
will still need to be tested/reviewed by someone else.

I can propose myself as a maintainer for docs if you are amenable.

Regards,
Gyan


More information about the ffmpeg-devel mailing list