ffmpeg-cli-wrapper icon indicating copy to clipboard operation
ffmpeg-cli-wrapper copied to clipboard

Add support for -i pipe:0

Open AlexanderSchuetz97 opened this issue 1 year ago • 7 comments
trafficstars

Hello, I need to transcode live videos, essentially reading from an InputStream and pipeing the output into an OutputStream is what I need to do. Unfortunately writing the file to disk is not possible for my use case. I tried using 2 unix sockets which did work for some formats, but some input formats did not work that way. notably the MP4 demuxer didnt like demuxing from a unix socket. It does however work with demuxing from stdin.

Sadly your CLI wrapper has no support for demuxing from stdin, so I have to rely on ProcessBuilder directly for this one task.

I wouldnt mind developing this as a feature and making a PR to this repo if you are intrested in this. My idea would be to add a function to FFmpegBuilder that accepts a supplier to a InputStream. A bit like this:

FFmpegBuilder builder = ...
Supplier<InputStream> streamSupplier = ...
builder.setStdIn(streamSupplier)

At some appropriate location in the library I would add this code or something similar: (This may possible require starting of a "background" thread and I will maybe have to make adjustments for java 8 compat.)

Supplier<InputStream> streamSupplier = ...
Process proc = ...
if (streamSupplier != null) {
try(OutputStream outputStream = proc.getOutputStream(); 
    InputStream stream = streamSupplier.get()) {
    stream.transferTo(outputStream);
}
}

After these changes someone could then call builder.addInput("pipe:0") to pipe the input from the InputStream. For my usecase the output would still be a unix socket since my output muxer can write to the unix socket and the input would be from the InputStream.

Tell me if you have any intrest in this feature and I will begin making it and give you a PR shortly.

AlexanderSchuetz97 avatar Jul 26 '24 14:07 AlexanderSchuetz97