Re: [Linuxsampler-devel] Load Instrument patch (2)

[Rui N. C. <rn...@rn...>]

hi benno,

> yeah I tried to apply your patch to the current CVS but hunks got
> rejected,
> I'll try again with this patch.

The former only applies to previous and nice working linuxsampler
snapshot, as you find on ls.org downloads section. This later applies to
CVS HEAD as of yesterday, at least.

> I did not try it yet but I took a peek at the patch and since you
> don't seem to do anything particularily hackish (which could upset
>  our dictator, aka cuse) :) I'd say if noone complains within 24h or so
>  just go ahead and commit it.

Depending on your notion of what's a hack, the patch introduces a new
class LSCPLoadInstrument, inherited from Thread, which the only purpose is
to spawn Engine::LoadInstrument() in background. No black hat hacker magic
;)

One thing I dislike is that self-delete on LSCPLoadInstrument::Main() last
line, but I think it seems safe for now.


> PS: regarding qsampler, liblscp  since I want to keep /usr/local clean
> I want to put qsampler and liblscp into my own dirs but the problem is
> when compiling qsampler you have to hack the configure script and/or
> the .pro file so that it detects the presence of liblscp and then sets
> the paths for -I and -L when compiling.
> Would it be possible to add a configure option to qsampler so that
> you can specify --liblscp-dir=/tmp/mydir or something like that ?
> It's not strictly needed but would make it a bit easier to keep
> liblscp/qsampler in different dirs eg, giving the possibility to the
> user to keep multiple version and compare them against each other for
> debugging purposes etc.
>

You can use --prefix=/whatever, but for all ABI purposes at hand wouldn't
it suffice to set LD_LIBRARY_PATH to where is that your bleeding edge
liblscp.so? Incidentally this is my preferred method for debugging, YMMV.

Nevertheless, a note has been taken for that --with-liblscp=dir configure
option.

But remember that I'm just a noob and also a complete autofool moron ;)
-- 
rncbc aka Rui Nuno Capela
rn...@rn...