Re: [PATCH] Re: INSTALLPATH_PKGINCLUDEDIR



   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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]