Re: [PATCH] Re: INSTALLPATH_PKGINCLUDEDIR

[Stefan Westerfeld <stefan space twc de>]

   Hi!

On Tue, Nov 03, 2015 at 11:52:31PM +0100, timj gnu org wrote:
> From: Tim Janik <timj gnu org>
>
> 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