Bug 122 - liblinuxsampler
Summary: liblinuxsampler
Status: CLOSED FIXED
Alias: None
Product: LinuxSampler
Classification: Unclassified
Component: other (show other bugs)
Version: SVN Trunk
Hardware: PC Linux
: P2 normal
Assignee: Christian Schoenebeck
URL:
Depends on:
Blocks:
 
Reported: 2009-03-28 16:19 CET by Luis Garrido
Modified: 2009-03-29 20:46 CEST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Luis Garrido 2009-03-28 16:19:53 CET
Hi!

To get liblinuxsampler to work I have to make two small changes:

1) I need to add atomic.h to the headers installed by src/common/Makefile.am,
since it is needed by engines/EngineChannel.h

2) This is more likely a distribution issue, no not sure if there is any change
to make at LS side... Perhaps at the .spec file, but I am using PlanetCCRMA's
spec instead, so I will address this with its maintainer too. Since
liblinuxsampler is installed in /usr/lib/linuxsampler I have to add that path to
the dynamic loader configuration, so I need to create
/etc/ld.so.conf.d/linuxsampler.conf containing LS lib path and run ldconfig.
Comment 1 Christian Schoenebeck 2009-03-28 17:39:32 CET
(In reply to comment #0)
> 1) I need to add atomic.h to the headers installed by 
src/common/Makefile.am,
> since it is needed by engines/EngineChannel.h

No, atomic.h should never be exposed to the API, since it's architecture 
specific and would in turn require to include config.h. Seems Grigor added 
those atomic_t variables accidently to EngineChanel.h. I will fix that in the 
next days, by hiding all private data of the EngineChannel on the heap. Then 
we don't need to have the atomic.h include there.

> 2) This is more likely a distribution issue, no not sure if there is any 
change
> to make at LS side... Perhaps at the .spec file, but I am using 
PlanetCCRMA's
> spec instead, so I will address this with its maintainer too. Since
> liblinuxsampler is installed in /usr/lib/linuxsampler I have to add that 
path to
> the dynamic loader configuration, so I need to create
> /etc/ld.so.conf.d/linuxsampler.conf containing LS lib path and run ldconfig.

Yes, that's in fact distribution specific. On most systems (e.g. on Debian 
based ones) this is not necessary.
Comment 2 Christian Schoenebeck 2009-03-29 20:44:50 CEST
Just commited it to CVS. Thanks for the report!