20 May 17:56
Re: [lttng-dev] [PATCH] configure: exit if urcu/uatomic.h is not found
On Sun, May 20, 2012 at 11:28:29AM -0400, Mathieu Desnoyers wrote: > * Christoph Hellwig (hch@...) wrote: > > We could have a local include/uatomic.h that includes either based on > > which one is available, still erroring out if none is present. > > > > I've cced the LTTng list to se if using the old uatomic_arch.h header > > from outside librcu is something we should do at all. > > Hi Christoph, > > Hrm, 0.4.1 is really old (2010-02-12). We did migrate some header names > and clean up some API namespacing back then between 0.4 and 0.5, so we > would not have to do it when the project would get more exposure. In > terms of way forward, we were planning to remove urcu/uatomic_arch.h > after a rather long deprecation period, being replaced by > urcu/uatomic.h. As you noticed, there has been a deprecation #warning in > place since the move has been done. > > If you want to keep compatibility with the old 0.4.x version of liburcu > for a longer time, my recommendation would be to add a configure.ac > check for urcu/uatomic.h. It it is missing, then check for > urcu/uatomic_arch.h. This would specify a preprocessor macro that will > select either in a wrapper header within the sheepdog project. > If you only rely on uatomic, doing so should handle deprecation of the > older liburcu 0.4.x support within sheepdog at some point in the future > if need be. Another approach is to have sheepdog simply download the desired version of liburcu if it doesn't like the one it finds.(Continue reading)![]()
RSS Feed