[FFmpeg-devel] [FFmpeg-cvslog] cmdutils: Add -buildconf option.

Stephen Hutchinson qyot27 at gmail.com
Wed Nov 27 04:19:55 CET 2013


On Tue, Nov 26, 2013 at 3:58 AM, Clément Bœsch <u at pkh.me> wrote:
> av_escape() won't help?
>
> I'm pretty sure we already deal with such issue. Try:
>   make libavutil/avstring-test
>   make tools/ffescape
> ...and look how they work.
>

I don't think it does, or at least not by itself.  It's either
escaping all spaces in the entire string or none of them, rather than
only the ones inside the substrings bounded by single-quotes.  That's
with either tools/ffescape or using av_escape directly.

It needs to look like:
=> "--prefix=/home/qyot27/ffmpeg_build --pkg-config=\'pkg-config\
--static\' --enable-gpl --enable-version3 --enable-avisynth
--extra-ldflags=\'-mconsole\ -Wl,--multiple-declaration\ --static\'
--cpu=pentium3 --extra-cflags=\'-mfpmath=sse\ -march=pentium3\ -msse\
-mtune=pentium3\'"

> I was just referring to the method used:
>
>   const int indent = 2;
>   av_log(NULL, level, "\n%*cconfiguration:\n", indent, ' ');
>   while (splitconf) {
>       av_log(NULL, level, "%*c%s\n", indent * 2, ' ', splitconf);
>
> ...which I find a bit more sexy. Consider this a nitpick.
>

Fixed locally.


More information about the ffmpeg-devel mailing list