Hi!

On Tue, Nov 03, 2015 at 11:52:31PM +0100, [email protected] wrote:
> From: Tim Janik <[email protected]>
> 
> Hey Stefan,
> 
> I'm not too happy to introduce another INCLUDE identifier when the existing
> one isn't really used.

Ok, I don't mind either way.

> Also the patch doesn't apply with the recent versioning
> changes Beast has seen. I think this patch should fix your problem well.
> 
> Tim Janik (1):
>   BUILD: fix BSE_INSTALLPATH_INCLUDES to point to the sfidl standard
>     includes
> 
>  bse/Makefile.am | 2 +-
>  configure.ac    | 4 +++-
>  sfi/Makefile.am | 2 +-
>  3 files changed, 5 insertions(+), 3 deletions(-)

No, it doesn't:

stefan@quadcorn:~/src/beast/sfi (master *% u=)$ ./sfidl --print-include-path
${prefix}/include/bse-0

This should not contain a ${prefix} at all, but point to the actual directory.
So there needs to be another string|variable dereferencing step in
configure.ac.

If you look at my original patch, its this change,

-libdir=`eval echo "${libdir}"` ; datarootdir=`eval echo "${datarootdir}"` ; 
datadir=`eval echo "${datadir}"`
+libdir=`eval echo "${libdir}"` ; datarootdir=`eval echo "${datarootdir}"` ; 
datadir=`eval echo "${datadir}"` ; includedir=`eval echo "${includedir}"`

plus a bit more code to backup and restore the original value of ${includedir}.

   Cu... Stefan
-- 
Stefan Westerfeld, http://space.twc.de/~stefan
_______________________________________________
beast mailing list
[email protected]
https://mail.gnome.org/mailman/listinfo/beast

Reply via email to