Re: [FFmpeg-devel] [PATCH 3/5] configure: improve gzip check

2025-05-26 Thread Jacob Lifshay
On Mon, May 26, 2025 at 6:34 PM Ramiro Polla wrote: > -[ -x "$(command -v gzip)" ] && enable gzip > +gzip -V > /dev/null 2>&1 && enable gzip idk if you intend it to build with busybox gzip, but busybox gzip doesn't support the -V option, a better command could be something like: gzip < /dev/null

Re: [FFmpeg-devel] [PATCH 3/5] configure: improve gzip check

2025-05-27 Thread Jacob Lifshay
On May 27, 2025 9:12:47 AM PDT, Ramiro Polla wrote: > Hi Jacob, > > Thanks. I hadn't thought about gzip implementations that don't support -V. > > What about the following? > echo | gzip -c > /dev/null 2>&1 && enable gzip That should also work afaik. busybox does support -c. Jacob _

[FFmpeg-devel] [PATCH] mpeg12enc: fix wrong size when copying A53_CC side-data into output

2025-05-15 Thread Jacob Lifshay
Fixes: https://trac.ffmpeg.org/ticket/11591 Signed-off-by: Jacob Lifshay --- libavcodec/mpeg12enc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c index 8364368fde..e48437eb8a 100644 --- a/libavcodec/mpeg12enc.c +++ b

Re: [FFmpeg-devel] [PATCH] mpeg12enc: fix wrong size when copying A53_CC side-data into output

2025-05-15 Thread Jacob Lifshay
On Thu, May 15, 2025 at 6:15 PM Andreas Rheinhardt wrote: > Thanks for this patch (and for the bug report; and sorry for causing you > trouble). Your patch was also my first attempt to fix this, but > unfortunately ff_copy_bits() may overread, yet the side data buffer need > not be padded, so my p

Re: [FFmpeg-devel] [PATCH 1/5] configure: remove build-time check for gzip support in zlib

2025-05-30 Thread Jacob Lifshay
On May 30, 2025 4:10:45 AM PDT, Ramiro Polla wrote: > I'll postpone this part of the patchset (patches 1 to 3) for a while > to give more time for people to comment. If anybody can find a system > where zlib doesn't support gzip, please let us know. idk if anyone is using it as the system zlib

Re: [FFmpeg-devel] what AVCodecID to use for copying full vanc data between .mxf and .mcc

2025-07-08 Thread Jacob Lifshay
On Tue, Jul 8, 2025 at 5:08 PM Devin Heitmueller wrote: > I took a quick look at your branch. So this is a good start, IMHO... thanks! > Note that I took a look at the MCC demuxer last year, and it's got a > bunch of issues where the parsing makes a bunch of assumptions (e.g. > doesn't filter o

[FFmpeg-devel] what AVCodecID to use for copying full vanc data between .mxf and .mcc

2025-07-07 Thread Jacob Lifshay
I'm currently writing a .mcc muxer, it currently translates from eia-608/708 to full vanc packets before outputting a .mcc file: https://github.com/programmerjake/FFmpeg/tree/add-mcc-mux I want to add the ability to the mxf and mcc muxers/demuxers to keep the full vanc data when doing stream cop

Re: [FFmpeg-devel] [POLL] [VOTE] code.ffmpeg.org

2025-07-13 Thread Jacob Lifshay
On July 13, 2025 4:43:57 AM PDT, Michael Niedermayer wrote: > Hi all > > Do people want Forgejo or Gitlab on code.ffmpeg.org for testing? i'll note that I ran into some important upsides and downsides with both of those when picking which to use for projects i've worked on, hopefully this

[FFmpeg-devel] [PATCH] lavu: add av_scanf_format() and use it on av_sscanf()

2025-07-20 Thread Jacob Lifshay
Signed-off-by: Jacob Lifshay --- doc/APIchanges | 3 +++ libavutil/attributes.h | 2 ++ libavutil/avstring.h | 2 +- libavutil/version.h| 2 +- 4 files changed, 7 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 80eb408788..5c40b7c13d 100644 --- a

Re: [FFmpeg-devel] FFmpeg 8.0 Release

2025-07-23 Thread Jacob Lifshay
On July 23, 2025 10:48:51 AM PDT, Michael Niedermayer wrote: > On Wed, Jul 23, 2025 at 06:43:51PM +0200, Dimitry Andric wrote: > > As long as there is a command line option to disable checking, it should > > be a good default. > > > There are many sites out there with badly configured > > cer

Re: [FFmpeg-devel] Notifications for the new monolithic tool

2025-07-23 Thread Jacob Lifshay
On July 23, 2025 3:06:40 PM PDT, Timo Rothenpieler wrote: > I've registered a dummy-user with the ffmpeg-devel address as its E-Mail and > subscribed it to all the repo notifications. > > So things will get forwared to the ML via it now. It seems to not be working, since you sent this I've got

Re: [FFmpeg-devel] what AVCodecID to use for copying full vanc data between .mxf and .mcc

2025-07-23 Thread Jacob Lifshay
On July 23, 2025 1:01:32 PM PDT, "Tomas Härdin" wrote: > Perhaps I should have made it clear that I'm just spitballing what kind > of workflows we should keep in mind with this stuff. I'm not suggesting > we need to add support for everything right away. Just that we should > take steps toward a s

Re: [FFmpeg-devel] [PATCH] web: announce code.ffmpeg.org

2025-07-22 Thread Jacob Lifshay
h the only reason I'm contributing now using git send-email is because I'm being paid to, if I weren't being paid I'm much less likely to have bothered even when I had the code in a git repo. Jacob Lifshay ___ ffmpeg-devel mailing list f

Re: [FFmpeg-devel] [PATCH] web: announce code.ffmpeg.org

2025-07-24 Thread Jacob Lifshay
On Thu, Jul 24, 2025 at 8:40 PM compn wrote: > people can always just manually attach patches to emails sent to > ffmpeg-devel list. no need for git send mail. ok, I prefer to never do that though because it makes reading the patches very annoying. Anyone should be able to read the patches withou

Re: [FFmpeg-devel] what AVCodecID to use for copying full vanc data between .mxf and .mcc

2025-07-22 Thread Jacob Lifshay
On July 22, 2025 5:21:24 AM PDT, "Tomas Härdin" wrote: > mån 2025-07-07 klockan 21:12 -0700 skrev Jacob Lifshay: > > I'm currently writing a .mcc muxer, it currently translates from eia- > > 608/708 to full vanc packets before outputting a .mcc file: > > http

Re: [FFmpeg-devel] [PATCH] [RFC] global/clang-format: Add .clang-format configuration for consistent formatting

2025-07-25 Thread Jacob Lifshay
Replying to softworkz from the mailing list archive: I tried out the .clang-format file you proposed and I ended up using: AlignConsecutiveDeclarations:Consecutive instead of: > +AlignConsecutiveDeclarations:AcrossEmptyLinesAndComments since otherwise clang-format will indent the f

[FFmpeg-devel] My pull request for adding MXF SMPTE 436m ANC and a mcc muxer is ready

2025-07-25 Thread Jacob Lifshay
Idk if forgejo notifies anyone when I make it no longer WIP, so I'm sending this email. https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024 Also, can libklvanc be added to the CI images? that PR uses libklvanc for some parts and it'd be nice if CI tested that. Jacob