From: Sergio Garcia Murillo <sergio.garcia.murillo@gmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] Patch which requires a new library Date: Thu, 7 Mar 2024 21:16:09 +0100 Message-ID: <CA+ag07Z2WWJGxmMYqsFK-VY5RLE0rQF_dQ2i3WiKLZnoYYHZ4Q@mail.gmail.com> (raw) In-Reply-To: <3878548.OAUNn9EPeQ@basile.remlab.net> On Thu, Mar 7, 2024 at 7:02 PM Rémi Denis-Courmont <remi@remlab.net> wrote: > Le torstaina 7. maaliskuuta 2024, 19.30.46 EET Sergio Garcia Murillo a > écrit : > > > The point is we don't want to use the external lib. > > > > For what? This is aws lib implementing the aws s3 signatures and using > > ffmpeg crypto libs. > > For not depending on an external library, especially one so small and so > specific that it is very unlikely to be shipped by the vast majority of > FFmpeg's downstreams. > That seems fair enough, however the downside is that sigv4 lib is supported and maintained by aws and it has a proper testing suite. > (sigv4 is just three thousand lines of C, and most of that is just > boilerplate.) > Would it be then acceptable to just copy/paste the library code (with ffmpeg coding style formatting) referencing the original library url in the code? Also, where should we include the sigv4 code, would libautil be the correct place? Best regards Sergio _______________________________________________ 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:[~2024-03-07 20:16 UTC|newest] Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-03-07 15:06 Sergio Garcia Murillo 2024-03-07 15:30 ` Kieran Kunhya 2024-03-07 17:16 ` Sergio Garcia Murillo 2024-03-07 17:20 ` Kieran Kunhya 2024-03-07 17:30 ` Sergio Garcia Murillo 2024-03-07 18:02 ` Rémi Denis-Courmont 2024-03-07 20:16 ` Sergio Garcia Murillo [this message] 2024-03-07 20:56 ` Kieran Kunhya
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=CA+ag07Z2WWJGxmMYqsFK-VY5RLE0rQF_dQ2i3WiKLZnoYYHZ4Q@mail.gmail.com \ --to=sergio.garcia.murillo@gmail.com \ --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