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 |