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

 bse/ | 2 +-    | 4 +++-
 sfi/ | 2 +-
 3 files changed, 5 insertions(+), 3 deletions(-)

No, it doesn't:

stefan quadcorn:~/src/beast/sfi (master *% u=)$ ./sfidl --print-include-path

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

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,

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