[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