[FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

Mats Peterson matsp888 at yahoo.com
Wed Dec 16 19:57:59 CET 2015


On 12/16/2015 07:52 PM, Mats Peterson wrote:
> On 12/16/2015 07:45 PM, Lou Logan wrote:
>> On Wed, 16 Dec 2015 19:18:33 +0100, Mats Peterson wrote:
>>
>>> If you can create a proper patchset out of the archive, you can remove
>>> that single line.
>>
>> That will not happen.
>>
>>> It's unnecessary for me to upload a new archive with such a small
>>> change. It's a whole new file, at that.
>>
>> Another reason nobody is looking at your patch(es) is that it has been
>> compressed; adding another step reviewers must take to accommodate your
>> non-standard methods of patch submission. Most people review via their
>> mail client, and if there is a patch that takes more effort than
>> usually required then it is more likely to be ignored because there are
>> always other patches to review and test.
>>
>> I looked at the contents of your archive and it contains four .diff
>> files and a single .c file. This is not possible to review or apply.
>> Please use git to make a proper patch as requested previously.
>>
>> See:
>> <http://www.ffmpeg.org/developer.html#Submitting-patches>
>> <http://www.ffmpeg.org/git-howto.html>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
> Haha... OK then. I've done what I can. Take it or leave it.
>
> Mats

I have said before that I'm not going to dwell into the complexities of 
Git, since it feels about as wasteful as programming a GUI in my book. 
If you can't accept the patches in their current format, so be it. 
Thanks for your time and understanding.

Mats

-- 
Mats Peterson
http://matsp888.no-ip.org/~mats/


More information about the ffmpeg-devel mailing list