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@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
-- 
sheepdog mailing list
[email protected]
http://lists.wpkg.org/mailman/listinfo/sheepdog

Reply via email to