| Message ID | 20200510190135.24015-1-andriy.gelman@gmail.com |
|---|---|
| State | Accepted |
| Commit | 6cd198860d6f63d53c9d25fec1e0650c3ae6a9da |
| Headers | show |
| Series | [FFmpeg-devel] fftools/ffmpeg: update text requesting samples | expand |
| Context | Check | Description |
|---|---|---|
| andriy/default | pending | |
| andriy/make | success | Make finished |
| andriy/make_fate | success | Make fate finished |
On Sun, 10. May 15:01, Andriy Gelman wrote: > From: Andriy Gelman <andriy.gelman@gmail.com> > > Signed-off-by: Andriy Gelman <andriy.gelman@gmail.com> > --- > > Same as commit d1e52e396b8aa778bd8d12bf25864beca0937d0a > > doc/developer.texi | 2 +- > fftools/cmdutils.c | 2 +- > fftools/ffmpeg.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/doc/developer.texi b/doc/developer.texi > index 51e7299b1d..b33cab0fc7 100644 > --- a/doc/developer.texi > +++ b/doc/developer.texi > @@ -625,7 +625,7 @@ If the patch fixes a bug, did you provide a verbose analysis of the bug? > If the patch fixes a bug, did you provide enough information, including > a sample, so the bug can be reproduced and the fix can be verified? > Note please do not attach samples >100k to mails but rather provide a > -URL, you can upload to ftp://upload.ffmpeg.org. > +URL, you can upload to @url{https://streams.videolan.org/upload/}. > > @item > Did you provide a verbose summary about what the patch does change? > diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c > index 7f5a5ca664..072589e358 100644 > --- a/fftools/cmdutils.c > +++ b/fftools/cmdutils.c > @@ -2217,7 +2217,7 @@ double get_rotation(AVStream *st) > if (fabs(theta - 90*round(theta/90)) > 2) > av_log(NULL, AV_LOG_WARNING, "Odd rotation angle.\n" > "If you want to help, upload a sample " > - "of this file to ftp://upload.ffmpeg.org/incoming/ " > + "of this file to https://streams.videolan.org/upload/ " > "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)"); > > return theta; > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c > index f697460a30..c86b413f73 100644 > --- a/fftools/ffmpeg.c > +++ b/fftools/ffmpeg.c > @@ -2370,7 +2370,7 @@ static int decode_video(InputStream *ist, AVPacket *pkt, int *got_output, int64_ > av_log(ist->dec_ctx, AV_LOG_WARNING, > "video_delay is larger in decoder than demuxer %d > %d.\n" > "If you want to help, upload a sample " > - "of this file to ftp://upload.ffmpeg.org/incoming/ " > + "of this file to https://streams.videolan.org/upload/ " > "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)\n", > ist->dec_ctx->has_b_frames, > ist->st->codecpar->video_delay); ping
On Fri, 29 May 2020, Andriy Gelman wrote: > On Sun, 10. May 15:01, Andriy Gelman wrote: >> From: Andriy Gelman <andriy.gelman@gmail.com> >> >> Signed-off-by: Andriy Gelman <andriy.gelman@gmail.com> >> --- >> >> Same as commit d1e52e396b8aa778bd8d12bf25864beca0937d0a >> >> doc/developer.texi | 2 +- >> fftools/cmdutils.c | 2 +- >> fftools/ffmpeg.c | 2 +- >> 3 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/doc/developer.texi b/doc/developer.texi >> index 51e7299b1d..b33cab0fc7 100644 >> --- a/doc/developer.texi >> +++ b/doc/developer.texi >> @@ -625,7 +625,7 @@ If the patch fixes a bug, did you provide a verbose analysis of the bug? >> If the patch fixes a bug, did you provide enough information, including >> a sample, so the bug can be reproduced and the fix can be verified? >> Note please do not attach samples >100k to mails but rather provide a >> -URL, you can upload to ftp://upload.ffmpeg.org. >> +URL, you can upload to @url{https://streams.videolan.org/upload/}. >> >> @item >> Did you provide a verbose summary about what the patch does change? >> diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c >> index 7f5a5ca664..072589e358 100644 >> --- a/fftools/cmdutils.c >> +++ b/fftools/cmdutils.c >> @@ -2217,7 +2217,7 @@ double get_rotation(AVStream *st) >> if (fabs(theta - 90*round(theta/90)) > 2) >> av_log(NULL, AV_LOG_WARNING, "Odd rotation angle.\n" >> "If you want to help, upload a sample " >> - "of this file to ftp://upload.ffmpeg.org/incoming/ " >> + "of this file to https://streams.videolan.org/upload/ " >> "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)"); >> >> return theta; >> diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c >> index f697460a30..c86b413f73 100644 >> --- a/fftools/ffmpeg.c >> +++ b/fftools/ffmpeg.c >> @@ -2370,7 +2370,7 @@ static int decode_video(InputStream *ist, AVPacket *pkt, int *got_output, int64_ >> av_log(ist->dec_ctx, AV_LOG_WARNING, >> "video_delay is larger in decoder than demuxer %d > %d.\n" >> "If you want to help, upload a sample " >> - "of this file to ftp://upload.ffmpeg.org/incoming/ " >> + "of this file to https://streams.videolan.org/upload/ " >> "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)\n", >> ist->dec_ctx->has_b_frames, >> ist->st->codecpar->video_delay); > > ping LGTM, thanks. Marton
On Sun, 31. May 19:16, Marton Balint wrote: > > > On Fri, 29 May 2020, Andriy Gelman wrote: > > > On Sun, 10. May 15:01, Andriy Gelman wrote: > > > From: Andriy Gelman <andriy.gelman@gmail.com> > > > > > > Signed-off-by: Andriy Gelman <andriy.gelman@gmail.com> > > > --- > > > > > > Same as commit d1e52e396b8aa778bd8d12bf25864beca0937d0a > > > > > > doc/developer.texi | 2 +- > > > fftools/cmdutils.c | 2 +- > > > fftools/ffmpeg.c | 2 +- > > > 3 files changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/doc/developer.texi b/doc/developer.texi > > > index 51e7299b1d..b33cab0fc7 100644 > > > --- a/doc/developer.texi > > > +++ b/doc/developer.texi > > > @@ -625,7 +625,7 @@ If the patch fixes a bug, did you provide a verbose analysis of the bug? > > > If the patch fixes a bug, did you provide enough information, including > > > a sample, so the bug can be reproduced and the fix can be verified? > > > Note please do not attach samples >100k to mails but rather provide a > > > -URL, you can upload to ftp://upload.ffmpeg.org. > > > +URL, you can upload to @url{https://streams.videolan.org/upload/}. > > > > > > @item > > > Did you provide a verbose summary about what the patch does change? > > > diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c > > > index 7f5a5ca664..072589e358 100644 > > > --- a/fftools/cmdutils.c > > > +++ b/fftools/cmdutils.c > > > @@ -2217,7 +2217,7 @@ double get_rotation(AVStream *st) > > > if (fabs(theta - 90*round(theta/90)) > 2) > > > av_log(NULL, AV_LOG_WARNING, "Odd rotation angle.\n" > > > "If you want to help, upload a sample " > > > - "of this file to ftp://upload.ffmpeg.org/incoming/ " > > > + "of this file to https://streams.videolan.org/upload/ " > > > "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)"); > > > > > > return theta; > > > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c > > > index f697460a30..c86b413f73 100644 > > > --- a/fftools/ffmpeg.c > > > +++ b/fftools/ffmpeg.c > > > @@ -2370,7 +2370,7 @@ static int decode_video(InputStream *ist, AVPacket *pkt, int *got_output, int64_ > > > av_log(ist->dec_ctx, AV_LOG_WARNING, > > > "video_delay is larger in decoder than demuxer %d > %d.\n" > > > "If you want to help, upload a sample " > > > - "of this file to ftp://upload.ffmpeg.org/incoming/ " > > > + "of this file to https://streams.videolan.org/upload/ " > > > "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)\n", > > > ist->dec_ctx->has_b_frames, > > > ist->st->codecpar->video_delay); > > > > ping > > LGTM, thanks. > Thanks, applied.
diff --git a/doc/developer.texi b/doc/developer.texi index 51e7299b1d..b33cab0fc7 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -625,7 +625,7 @@ If the patch fixes a bug, did you provide a verbose analysis of the bug? If the patch fixes a bug, did you provide enough information, including a sample, so the bug can be reproduced and the fix can be verified? Note please do not attach samples >100k to mails but rather provide a -URL, you can upload to ftp://upload.ffmpeg.org. +URL, you can upload to @url{https://streams.videolan.org/upload/}. @item Did you provide a verbose summary about what the patch does change? diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index 7f5a5ca664..072589e358 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@ -2217,7 +2217,7 @@ double get_rotation(AVStream *st) if (fabs(theta - 90*round(theta/90)) > 2) av_log(NULL, AV_LOG_WARNING, "Odd rotation angle.\n" "If you want to help, upload a sample " - "of this file to ftp://upload.ffmpeg.org/incoming/ " + "of this file to https://streams.videolan.org/upload/ " "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)"); return theta; diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index f697460a30..c86b413f73 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -2370,7 +2370,7 @@ static int decode_video(InputStream *ist, AVPacket *pkt, int *got_output, int64_ av_log(ist->dec_ctx, AV_LOG_WARNING, "video_delay is larger in decoder than demuxer %d > %d.\n" "If you want to help, upload a sample " - "of this file to ftp://upload.ffmpeg.org/incoming/ " + "of this file to https://streams.videolan.org/upload/ " "and contact the ffmpeg-devel mailing list. (ffmpeg-devel@ffmpeg.org)\n", ist->dec_ctx->has_b_frames, ist->st->codecpar->video_delay);