Boost C++ Libraries: Ticket #7336: BOOST_THREAD_DONT_USE_SYSTEM doesn't work.

owner, status, milestone changed

viboes — Sat, 08 Sep 2012 16:02:28 GMT

owner changed from Anthony Williams to viboes
status new → assigned
milestone Boost 1.52.0 → To Be Determined

I'm sorry. I think that I wrote this documentation with the expectation that I would be nice to provide it and I didn't it at all.

I have take a look at the dependency and it will be quite complex to remove it for the next release. If I don't reach to remove the dependency for 1.52 I will remove this macro and state clearly that the dependency is needed.

There is an alternative that is to define the Boost.System undocumented BOOST_ERROR_CODE_HEADER_ONLY macro, but this is yet bugged (See #7347 duplicate symbol while BOOST_ERROR_CODE_HEADER_ONLY is defined).

viboes — Sat, 08 Sep 2012 16:03:34 GMT

BTW, do you want to remove the dependency on Boost.System or the need to link with?

Sat, 08 Sep 2012 17:50:31 GMT

Thank you for your answer and your work on this issue.

Replying to viboes:

BTW, do you want to remove the dependency on Boost.System or the need to link with?

To me, a clear statement that I have to link with Boost.System (or do whatever that library requires) to work with Boost.Thread is enough.

Also, BOOST_THREAD_DONT_USE_SYSTEM, if implemented, would be a switch that changes whole library ABI (exception types), which is scaring a bit.

viboes — Sat, 08 Sep 2012 20:56:59 GMT

Replying to Ivan A. Melnikov <iv@…>:

Thank you for your answer and your work on this issue.

Replying to viboes:

BTW, do you want to remove the dependency on Boost.System or the need to link with?

To me, a clear statement that I have to link with Boost.System (or do whatever that library requires) to work with Boost.Thread is enough.

Great. I will fix the documentation.

Also, BOOST_THREAD_DONT_USE_SYSTEM, if implemented, would be a switch that changes whole library ABI (exception types), which is scaring a bit.

Yes, now that Boost.Thread uses Boost.Chrono and the interface for the exceptions and the error codes in futures follows the C++11 standard, disabling all this stuff will represent quite a lot of work without no clear gain.

In addition, removing the dependency to Boost.System implies removing the dependency to Boost.Chrono.

I hope that #7347 will be fixed soon so that the link to Boost.System and Boost.Chrono could be optional.

milestone changed

viboes — Sat, 08 Sep 2012 21:00:36 GMT

milestone To Be Determined → Boost 1.52.0

attachment set

viboes — Sun, 09 Sep 2012 11:08:52 GMT

attachment → 7336.patch

Documentation fix

viboes — Sun, 09 Sep 2012 12:36:32 GMT

Committed in trunk revision [80459].

status changed; resolution set

viboes — Thu, 13 Sep 2012 19:12:20 GMT

status assigned → closed
resolution → fixed

Committed revision 80516.