[FFmpeg-devel] [PATCH] avfilter/image2: Add source file path and basename to each packet side data.
Alexandre Heitor Schmidt
alexandre.schmidt at gmail.com
Sat Dec 21 18:53:40 EET 2019
Sorry, Nicolas. I'm new here. I'll keep the mails short for sure.
Hopefully my next submissions won't take this long to process, as now I
know more about how the entire thing works. Even reading the docs
regarding patch submissions (which I did), there are things that I had
to learn by trial & error until I do it exactly how they're supposed to
be done.
To answer your questions: yes, I've run make fate and built ffmpeg
completely. But specifically for this problem with the HTML docs, the
issue was I didn't have texi2html installed, so the build process wasn't
generating any of the HTML docs (CONFIG_HTMLPAGES = 0), but no warnings
or similar were being shown. For the man pages, those non-escaped "{}"
didn't raise any errors, so I thought it was okay.
Anyway, I hope it gets through now. Everything re-tested.
Thank you for the hints.
Alex.
On 21/12/2019 16:34, Nicolas George wrote:
> Alexandre Heitor Schmidt (12019-12-21):
>> I fixed it again. Some {} weren't escaped by the "@" character, thus causing
>> this error.
> How did this happen? Did you not build ffmpeg completely (and run FATE)
> before sending the patch?
>
> Also, could you please trim your mails? >250 lines of quote for just 3
> new lines is not a good signal/noise ratio, especially since your mailer
> re-wraps quotes (which it should not).
>
> Regards,
More information about the ffmpeg-devel
mailing list