Re: [Gnash-dev] Re: CPP undefined: broke in 11080

[strk]

On Fri, Jun 12, 2009 at 11:16:51AM +0200, strk wrote:
> On Fri, Jun 12, 2009 at 11:03:37AM +0200, strk wrote:
> 
> > There's no obvious reason why CPP should get
> > undefined after the changes in configure.ac,
> > which basically just add a call to GNASH_PKG_FIND
> > for SSL ...
> 
> Indeed dropping that call fixes the issue.
> 
> Maybe it's too early in the configure.ac file
> for using GNASH_PKG_FIND, as it's the first one.

Eh... I found this later in the configure.ac file:

dnl !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
dnl !!
dnl !! IMPORTANT NOTICE
dnl !!
dnl !!  Any call to GNASH_PATH_XXX must be be given *after* this snippet.
dnl !!  This is to ensure that PKG_CONFIG, cross_compiling and incllist are
dnl !!  properly set at the time of call.
dnl !!
dnl !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Moving the GNASH_PKG_FIND call after that block fixed the bug.
Committed as revision 11086 with an addition to that NOTICE
talking explicitly about GNASH_PKG_FIND too.

Note that there are other GNASH_PATH_* uses before that block,
not too far. Might be wise to move them down too.

--strk;

 Free GIS & Flash consultant/developer      ()  ASCII Ribbon Campaign
 http://foo.keybit.net/~strk/services.html  /\  Keep it simple!