On Monday 20 August 2012 21:55:55 Vicente J. Botet Escriba wrote:
Le 20/08/12 18:44, Andrey Semashev a écrit :
I think Boost.Thread always depends on Boost.DateTime, at least on Windows. I see it includes Boost.DateTime in src/win32/thread.cpp.
Yes, but it uses only inline functions.
I think, the dependency on Boost.DateTime should be in the Jamfile of Boost.Thread. This way the build system will know that Boost.DateTime should also be compiled when Boost.Thread builds.
An the dependency in on inline functions, I suspect that others than you will request that Boost.Thread don't link with Boost.DateTime. Note that currently, everything is OK without linking with.
Well, I would add the dependency anyway. I didn't verify but doesn't MSVC auto-link Boost.DateTime to Boost.Thread just because of the header include? Anyway, if you're certain you want to avoid the dependency then there are two solutions. The simplest one, as you suggested, is to add a note to the docs. Another solution is to make Boost.DateTime inclusion optional, much like it is with Boost.Chrono now. I'm personally fine with either one.