Boost C++ Libraries: Ticket #10637: Certain thread unit tests fail to compile on Xcode 6 because BOOST_NO_CXX11_SMART_PTR is not defined

severity changed

nat@… — Wed, 08 Oct 2014 19:11:44 GMT

severity Problem → Regression

This appears to be a regression. Although asyncasync_executor_p (async_executor_pass.cpp) does not appear in Boost 1.55.0, the other two unit tests pass in Boost 1.55.0.

viboes — Wed, 08 Oct 2014 19:22:19 GMT

Please, could you tell me with which version, platform, compiler and C++ lib these error appear?

nat@… — Wed, 08 Oct 2014 19:29:46 GMT

Replying to viboes:

Please, could you tell me with which version, platform, compiler and C++ lib these error appear?

version: Boost 1.56.0

platform: Mac OS X 10.9.5

g++ --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/include/c++/4.2.1
Apple LLVM version 6.0 (clang-600.0.51) (based on LLVM 3.5svn)
Target: x86_64-apple-darwin13.4.0

library: thread

viboes — Wed, 08 Oct 2014 20:22:14 GMT

I suspect that you are using option c++0x and libstdcpp. This is not supported.

nat@… — Wed, 08 Oct 2014 20:37:25 GMT

Replying to viboes:

I suspect that you are using option c++0x and libstdcpp. This is not supported.

I pasted the b2 command line I used in the Description. How should I modify that command line?

owner, status changed

viboes — Wed, 08 Oct 2014 21:29:38 GMT

owner changed from Anthony Williams to viboes
status new → assigned

Oh, I'm really sorry. I believed that you were using clang.

It seems that default_delete is not movable with your default library. I suggest you to do the following.

In boost/thread/csbl/memory/unique_ptr.hpp, there is the following check

// 20.8.1 class template unique_ptr:
#if defined BOOST_NO_CXX11_SMART_PTR

You could define in your application BOOST_NO_CXX11_SMART_PTR

BTW, there are no problems inhttp://www.boost.org/development/tests/master/developer/thread.html for Sandia-darwin-4.2.1 Please, could you tell me if it works on develop or master branch, so that I can see what I can do if it is not the case?

P.S. in version 1.57 I'm using always movelib::unique_ptr.

viboes — Wed, 08 Oct 2014 21:35:55 GMT

After more thoughts, you should be compiling with c++98 and you have unique_ptr. This is a mess.

nat@… — Wed, 08 Oct 2014 22:58:56 GMT

Replying to viboes:

You could define in your application BOOST_NO_CXX11_SMART_PTR

Please, could you tell me if it works on develop or master branch, so that I can see what I can do if it is not the case?

Thank you! Adding BOOST_NO_CXX11_SMART_PTR to the b2 command line does indeed permit these tests to pass:

./b2 toolset=darwin variant=release address-model=32 architecture=x86 -sNO_BZIP2=1 cxxflags=-DBOOST_NO_CXX11_SMART_PTR cxxflags=-Wno-c99-extensions cxxflags=-Wno-variadic-macros cxxflags=-Wno-unused-function cxxflags=-Wno-unused-const-variable libs/thread/test//async__async_p libs/thread/test//async__async_executor_p libs/thread/test//promise__set_rvalue_at_thread_exit_p

component, summary changed

viboes — Sat, 11 Oct 2014 03:03:25 GMT

component thread → config
summary Certain thread unit tests fail to compile on Xcode 6 → Certain thread unit tests fail to compile on Xcode 6 because BOOST_NO_CXX11_SMART_PTR is not defined

So resuming you are using the following command line with c++98 and BOOST_NO_CXX11_SMART_PTR is not defined.

./b2 toolset=darwin variant=release address-model=32 architecture=x86 -sNO_BZIP2=1 cxxflags=-DBOOST_NO_CXX11_SMART_PTR cxxflags=-Wno-c99-extensions cxxflags=-Wno-variadic-macros cxxflags=-Wno-unused-function cxxflags=-Wno-unused-const-variable libs/thread/test//async__async_p libs/thread/test//async__async_executor_p libs/thread/test//promise__set_rvalue_at_thread_exit_p

Moving it to Boost.Config then.

John Maddock — Mon, 23 Feb 2015 13:01:26 GMT

Sorry to be late coming to this - that macro should be set for all GCC-4.3 and earlier (including your 4.2.x install) in boost/config/stdlib/libstdcpp3.hpp. Can you see why the branch starting:

#if (BOOST_LIBSTDCXX_VERSION < 40400)

!defined(BOOST_LIBSTDCXX11)

Is not being taken on your system?

Thanks, John.

nat@… — Mon, 23 Feb 2015 18:30:28 GMT

I'm sorry. At some point a couple months ago the config did start defining that symbol, so I removed our explicit definition because it was producing a ton of duplicate-macro warnings. I've reviewed our changelog. The only change (between introducing BOOST_NO_CXX11_SMART_PTR and removing it again) that suggests itself as a possible cause is that we set the Mac OS X SDK to 10.9. I confess I don't know what effect that has on the set of predefined macros.

Shortly after removing BOOST_NO_CXX11_SMART_PTR, we updated our Boost source to 1.57. Notably, though, that changeset was committed after the changeset removing BOOST_NO_CXX11_SMART_PTR.

In any case, I should have updated this ticket to note that the problem no longer manifests for us.

status changed; resolution set

John Maddock — Mon, 23 Feb 2015 19:22:19 GMT

status assigned → closed
resolution → obsolete

OK thanks, closing down, please reopen if it comes back again or you can provide more info.