Le septidi 7 brumaire, an CCXXIII, Binathi a écrit : > Branch: refs/heads/master > Home: https://github.com/Binathi/ffmpeg > Commit: d83ead999c0343b74d66b58d02c3ef381ac43b69 > > https://github.com/Binathi/ffmpeg/commit/d83ead999c0343b74d66b58d02c3ef381ac43b69 > Author: Binathi <binti...@gmail.com> > Date: 2014-10-28 (Tue, 28 Oct 2014) > > Changed paths: > M doc/ffserver.conf > M ffserver.c > > Log Message: > ----------- > Forks new process to allow ffserver run in daemon mode
I am sorry, but that it still not correct. You uploaded a snapshot of the source code to the tree, that makes it completely distinct from official FFmpeg repository. You need to make a clone of the official repository and work your patches in a branch started from the head. Then you must produce a patch and send it to the mailing list, using git format-patch or git send-email. I strongly suggest that just before actually sending the patch, you read it yourself carefully, and you compare it to patches sent to the mailing list that are quickly accepted. If your patch is very different from the ones on the mailing list, there is probably something you need to fix. In this particular instance, I see that your pull request has "3,894 additions, 4,389 deletions" on ffserver.c: this is way too much for such a small feature, or for any reasonable feature for that matter. It seems you lost all the code's indentation somewhere, and the thousands of changes are mostly that. You would probably be better of re-starting with a completely new clone of the official repository (I insist: clone; do not download any source code using your browser), and then use the -w option of diff to help you import manually your changes from the first working tree. Regards, -- Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
