From: Nicolas George <george@nsup.org> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] API enhancements / broken promises Date: Wed, 17 Aug 2022 22:48:10 +0200 Message-ID: <Yv1UClbvFzy3dDbC@phare.normalesup.org> (raw) In-Reply-To: <20220817172145.GD2088045@pb2> [-- Attachment #1.1: Type: text/plain, Size: 4831 bytes --] Hi. Michael Niedermayer (12022-08-17): > As i said elsewhere i think replacing BPrint by AVWriter is a good idea. > and also that the TC does to the best of my knowledge have no power in > this case. We first need some patch and disagreement the TC could vote > on. ATM there is no public disagreement on a patch i think > > But i didnt hit reply to repeat that. Thank you and thank Stefano for your encouraging replies. With them and the comments I had in private and/or earlier, I am beginning to feel confident I can override the argument-less objection. Note that I am not asking for a blank check to push anything I want. I will of course submit the code for review and adhere to the technical comments made to it. And I want to insist, not just for me: this is a mechanism we need. Right now, the only way to know if a change will be accepted is to submit a patch series in its almost final shape. For something ambitious, that can mean a lot of work, and if the patches are eventually rejected, that work goes to waste. The possibility that it happens is discouraging. And with that discouragement, we only get low-hanging fruits, changes that are sure to be accepted. Or we get intimidation: submit a 100+ patch series, present the project with a fait accopli, and count on the fact that nobody will dare object in the face of the sheer amount of work, even if the proposal is flawed at its core and would need a complete rewrite. The lack of a procedure to engage in something ambitious without the risk of outright rejection is limiting the progress of FFmpeg, and it is sad. Maybe this simple idea would be enough, if people like it: doc/plans.md containing more-or-less detailed plans of ambitious plans. Patches to doc/plans.md are reviewed on the mailing list like any other patches. But once a plan is accepted, patches that implement it cannot be objected on principle by naysayers and bikeshedders, only on technical matters. I will try to propose a small patch for that, to get the conversation going. > Rather i wanted to comment on > XML and JSON and FFmpeg. Thanks. > I saw on IRC this (authors removed because their names are not important here, > iam really replying to the statments not the people) > A> but I, IMVHO, dont think this project should get a XML parser > B> "nice and limited" XML parser sounds like all sorts of "fun" > C> Yeah, xml/json/... parser in ffmpeg is just... no > > Whats the purpose of FFmpeg ? > "A complete, cross-platform solution to record, convert and stream audio and video." > first actually i think that should be chnaged to something like this: > "A complete, cross-platform solution to your multmedia needs" > Because there are subtitles and many other things we do care about that this > misses > > Now to achieve this do we need xml and json ? > grep tells me we have 500 matches (not counting docs) for xml and almost 100 > for json > Also for streaming and some cases filtering being able to serialize objects > would be useful. xml and json seem better choices than some ad-hoc format > So i would awnser the question do we need XML and JSON, with yes we live > in a world that uses XML and JSON so if we give the option to use it too > that makes it easier for others to interact. > > now do we need our own implementation of it ? I dont know but we have > in almost all cases favored our native implementations when someone wrote > one. And libxml2 has had so many security issues that i think we should > at least consider replacing it. Replacing our use of libxml2 was my reason for starting to think and then write a XML parser, for exactly the reasons we describe, plus the fact that libxml2 is not a system library. We need XML for at least MPEG-DASH and IMF, which are unambiguously part of FFmpeg's purpose. And the objective of the "limited" parser would be to be able to digest DASH manifests found in the wild. > The 2nd thing that comes to mind with "purpose of FFmpeg" is FF stands for > "Fast Forward". To move fast forward we should not lose developers because of > rather minor disagreements. > IMO if Nicolas wants to write and maintain AVWriter and a simple xml parser > (which i presume has the goal to parse the XML we would generate for > serialization and in other places). I really think telling Nicolas "no" is > a unwise choice. But if someone is against very basic xml or json parsers > please speak up now and here because its still better to say "no" now than > after nicolas did the work. Thanks again. It is exactly my concern. If we collectively decide we do not want some project, I can work on something else. But the uncertainty is killing my motivation. Regards, -- Nicolas George [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2022-08-17 20:48 UTC|newest] Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-15 16:47 Nicolas George 2022-08-16 23:16 ` Stefano Sabatini 2022-08-17 17:21 ` Michael Niedermayer 2022-08-17 20:48 ` Nicolas George [this message] 2022-08-18 8:48 ` Tomas Härdin 2022-08-18 17:19 ` Jean-Baptiste Kempf 2022-08-19 18:30 ` Michael Niedermayer 2022-08-19 19:35 ` Timo Rothenpieler
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=Yv1UClbvFzy3dDbC@phare.normalesup.org \ --to=george@nsup.org \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git