[Sheepdog] [PATCH] configure: add urcu into configure

Liu Yuan namei.unix at gmail.com
Tue May 8 17:47:31 CEST 2012


On 05/08/2012 11:38 PM, Christoph Hellwig wrote:
> On Tue, May 08, 2012 at 11:31:53PM +0800, Liu Yuan wrote:
>> Yes, the latest urcu code provide a .pc file and it seems that the atomic header structure get
>> changed over time. Let's stick to the latest new structure and would better write a script to auto
>> download, compile and install it or point people to where he can finds the latest version to compile.
> 
> Not being able to use commonly packaged versions of dependencies is
> a major pain for anyone using a package.  So far it seems like the only
> problem is the configure script.  Unless we really need newer features
> inside the core code I'd really prefer allowing older versions.  I'll
> volunteer to add configure support for it.

It is nice of you to cook a patch for it. But be careful, I have tested it on
my not that new version Ubuntu (10.10),

yliu at yliu:~$ apt-cache showpkg liburcu-dev
Package: liburcu-dev
Versions: 
0.4.6-1 

No .pc config file *and* I looked at the atomic header structure 
uatomic_arch.h is the actual header we should include in the source file.

And it seems that even with newest liburcu, we should also include this file
instead of 'urcu/uatomic.h" to get backward compatible.

I am not sure what version comes up in most command server edition, such as
RHEL 6 and Debian stable, where we should support as well as possible. 
-- 
thanks,
Yuan



More information about the sheepdog mailing list