Bug 122

Summary: liblinuxsampler
Product: LinuxSampler Reporter: Luis Garrido <luisgarrido>
Component: otherAssignee: Christian Schoenebeck <cuse>
Status: CLOSED FIXED    
Severity: normal    
Priority: P2    
Version: SVN Trunk   
Hardware: PC   
OS: Linux   

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!