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

[Stephen Hutchinson]

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.