[FFmpeg-devel] [PATCH] Fix --extra-cflags (broken in r19411)

Måns Rullgård mans
Mon Jul 20 21:03:37 CEST 2009


Art Clarke <aclarke at xuggle.com> writes:

> In r19411, the --extra-cflags configure flag was broken if/when you passed
> in multiple invocations of the flag.  Prior versions of FFmpeg this flag was
> additive.  Now, only the last value wins.  To produce the problem, do this:
>
> configure --extra-cflags -I/directory1 --extra-cflags -I/directory2
>
> This patch reverts that change, but I'm sure it doesn't fix whatever the
> original committer was trying to fix.  That said, no ffmpeg-devel review was
> done of the original patch, and the check-in note for r19411 doesn't say WHY
> the checkin was done, so I'll leave it to the author to figure out how to
> deal with it.

There was of course a reason for that change, which you can see in r19465.

I have restored the old behaviour of the option in a proper way.
Thanks for reporting the issue.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list