Hello Hendrik, Thank you very much for the review, we will try to split one big invalid patch to sequence of small valid changes.
Tuesday, June 30, 2015, 12:03:10 AM, you wrote: HL> On Mon, Jun 29, 2015 at 10:57 PM, Ivan Uskov <ivan.us...@nablet.com> wrote: >> Hello All, >> >> I'm software developer of Nablet GmbH. We were requested by Intel to >> integrate all possible QSV-accelerated codecs and VPP modules into ffmpeg. >> This is my first commit which adding the QSV MPEG2 video decoder codec, I >> hope I did all or at least all most right. >> The 0001-Patch-for-QSV-components-by-Intel-applied.patch was received >> from Intel, so unfortunately it is not possible to split it to >> separate small patches. >> HL> The first patch is full of errors. It reverts changes from ffmpeg HL> mainline, it contains a dupe file (configure.orig), it contains HL> non-GPL compatible license headers. HL> Its not possible to apply it in any form unless it is severly cleaned HL> up and split - no matter if its from Intel or anyone, clean patches HL> are mandatory. HL> - Hendrik HL> _______________________________________________ HL> ffmpeg-devel mailing list HL> ffmpeg-devel@ffmpeg.org HL> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel -- Best regards, Ivan mailto:ivan.us...@nablet.com _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel