[FFmpeg-devel] [PATCH 1/7] build: fix objcc header check

[Clément Bœsch]

On Wed, Sep 06, 2017 at 10:05:40PM +0200, Thilo Borgmann wrote:
> Hi,
> 
> Am 06.09.17 um 11:58 schrieb Clément Bœsch:
> > From: Clément Bœsch <cboesch at gopro.com>
> > 
> > $headers is a variable set in the context of other functions (we don't
> > use the "local" keyword in our scripts, so those variables are global).
> > 
> > Currently, when checking for AVFoundation/AVFoundation.h, the actual
> > enabled header is math.h.
> > 
> > Similarly, when testing for QuartzCore/CoreImage.h, the actual enabled
> > header is CoreGraphics/CoreGraphics.h.
> > 
> > This is completely broken and may be the reason why these checks are
> > made in random places.
> 
> thus it was just a typo like the patch 1 suggested?
> 

What do you mean by "like the patch 1 suggested"? This is the patch 1.
And yeah, I guess it's a typo from day 1 (a2c6a99ba5). I didn't check if
the code was valid back then thanks to a side effect (basically if
check_cc was setting headers=$1 somewhere), but it's definitely broken
currently.

> Patch 1-5 ok!
> For patch 6-7 maybe someone else can have a closer look.

I'll wait a little more and push the patchset.

Thanks

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170907/8661a2d8/attachment.sig>