Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Paul B Mahol <onemda@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v3 0/8] add fd protocol
Date: Wed, 11 Jan 2023 09:52:08 +0100
Message-ID: <CAPYw7P4V23nFna5Vh4YEew-s6WVsv1RdB6qF8aUa90PPfzSq0A@mail.gmail.com> (raw)
In-Reply-To: <tencent_A6AB9DE4E1CC5AD43487812E34353E6D5709@qq.com>

On 1/11/23, "zhilizhao(赵志立)" <quinkblack@foxmail.com> wrote:
>
>
>> On Jan 11, 2023, at 15:53, Xiang, Haihao
>> <haihao.xiang-at-intel.com@ffmpeg.org> wrote:
>>
>> On Wo, 2023-01-11 at 11:18 +0800, zhilizhao(赵志立) wrote:
>>>> On Jan 9, 2023, at 11:17, zhilizhao(赵志立) <quinkblack@foxmail.com> wrote:
>>>>
>>>>> On Dec 15, 2022, at 01:10, Zhao Zhili <quinkblack@foxmail.com> wrote:
>>>>>
>>>>> From: Zhao Zhili <zhilizhao@tencent.com>
>>>>>
>>>>> v2:
>>>>> 1. Forbid pass file descriptor via fd:{num}, instead of -fd option
>>>>> 2. Set CLOEXEC
>>>>> 3. Prefer fd over pipe for fftools
>>>>>
>>>>> v1: dup file descriptor
>>>>>
>>>>> Zhao Zhili (8):
>>>>> avformat/file: add fd option for pipe
>>>>> avformat/file: reindent after the previous commit
>>>>> avformat/file: dup file descriptor for pipe
>>>>> avformat/file: add fd protocol
>>>>> fftools/ffmpeg_demux: disable stdin interaction for fd protocol
>>>>> fftools/ffplay: prefer fd over pipe for seek support
>>>>> fftools/ffprobe: prefer fd over pipe for seek support
>>>>> fftools/ffmpeg_demux: prefer fd over pipe for seek support
>>>>>
>>>>> doc/protocols.texi      |  31 +++++++-
>>>>> fftools/ffmpeg_demux.c  |   3 +-
>>>>> fftools/ffplay.c        |   2 +-
>>>>> fftools/ffprobe.c       |   4 +-
>>>>> libavformat/Makefile    |   1 +
>>>>> libavformat/file.c      | 152 +++++++++++++++++++++++++++++++---------
>>>>> libavformat/protocols.c |   1 +
>>>>> libavformat/version.h   |   4 +-
>>>>> 8 files changed, 156 insertions(+), 42 deletions(-)
>>>>
>>>> Will apply soon unless there are objections.
>>>
>>> Pushed.
>>
>> It caused link error on Windows
>>
>> INFO  - LD    libavformat/avformat-59.dll
>> INFO  -    Creating library libavformat/avformat.lib and object
>> libavformat/avformat.exp
>> INFO  - file.o : error LNK2019: unresolved external symbol S_ISREG
>> referenced in
>> function fd_open
>> INFO  - file.o : error LNK2019: unresolved external symbol S_ISBLK
>> referenced in
>> function fd_open
>> INFO  - libavformat\avformat-59.dll : fatal error LNK1120: 2 unresolved
>> externals
>
> Sorry for the break, I’m trying to figure out how to make it compatible
> with Windows.

Is this even portable?

>
>>
>> Thanks
>> Haihao
>>
>>>
>>>> _______________________________________________
>>>> 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".
>>>
>>> _______________________________________________
>>> 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".
>> _______________________________________________
>> 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".
>
> _______________________________________________
> 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".
>
_______________________________________________
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".

  reply	other threads:[~2023-01-11  8:52 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <34df4f8e-0c8b-4199-8a4b-5669078b4a25@EX-SZ069.tencent.com>
2023-01-09  3:17 ` "zhilizhao(赵志立)"
2023-01-11  3:18   ` "zhilizhao(赵志立)"
2023-01-11  7:53     ` Xiang, Haihao
2023-01-11  8:31       ` "zhilizhao(赵志立)"
2023-01-11  8:52         ` Paul B Mahol [this message]
2023-01-11  9:10           ` "zhilizhao(赵志立)"
2023-01-12  1:26             ` Xiang, Haihao
2023-01-12  7:29               ` "zhilizhao(赵志立)"
2023-01-12 19:13           ` Rémi Denis-Courmont
2023-01-13  3:37             ` "zhilizhao(赵志立)"
2023-01-13 18:12               ` Rémi Denis-Courmont
2023-01-11  9:02         ` "zhilizhao(赵志立)"
2022-12-14 17:10 Zhao Zhili

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=CAPYw7P4V23nFna5Vh4YEew-s6WVsv1RdB6qF8aUa90PPfzSq0A@mail.gmail.com \
    --to=onemda@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