Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

BoostPart building failed with clang 3.4 #864

Closed
Chilledheart opened this Issue · 3 comments

2 participants

@Chilledheart

When I am building YCM from the HEAD ( i.e. fa1d628 ), I got something wrong:

In file included from /Users/chilledheart/.vim/bundle/YouCompleteMe/cpp/BoostParts/libs/atomic/src/lockpool.cpp:3:
In file included from /Users/chilledheart/.vim/bundle/YouCompleteMe/cpp/BoostParts/boost/atomic.hpp:12:
In file included from /Users/chilledheart/.vim/bundle/YouCompleteMe/cpp/BoostParts/boost/atomic/atomic.hpp:17:
In file included from /Users/chilledheart/.vim/bundle/YouCompleteMe/cpp/BoostParts/boost/atomic/detail/platform.hpp:22:
/Users/chilledheart/.vim/bundle/YouCompleteMe/cpp/BoostParts/boost/atomic/detail/gcc-atomic.hpp:961:64: error: no matching constructor for
      initialization of 'storage_type' (aka 'boost::atomics::detail::storage128_type')
    explicit base_atomic(value_type const& v) BOOST_NOEXCEPT : v_(0)
                                                               ^  ~

Besides, I am using osx 10.9.2 with a clang-3.4 compiler. However it works fine with my ubuntu.more compiler errors.

It might be a known platform-related bug with boost, and I found the similar situation in macport and boost.Those patches (boostorg/atomic@6bb71fd and boostorg/atomic@e4bde20) mentioned in the first post work with me.

If you face the same situation, perhaps this can help you.

@Valloric
Owner

Thanks for reporting this! Since Boost upstream has already chose to apply these patches, I'll apply them to YCM's copy of Boost as well. They should be rolled into Boost 1.56 but until then, we need a fix for this.

@Valloric Valloric closed this issue from a commit
@Valloric Fixing Boost build with Clang 3.4
This applies the following Boost patches that have already been applied
upstream:

https://github.com/boostorg/atomic/commit/6bb71fdd8f7cc346d90fb14beb38b7297fc1ffd9.patch
https://github.com/boostorg/atomic/commit/e4bde20f2eec0a51be14533871d2123bd2ab9cf3.patch

These patches will be in Boost 1.56 but until then we need to make the local
changes.

Fixes #864
1882929
@Valloric Valloric closed this in 1882929
@Valloric
Owner

Please verify that you can build vanilla YCM now with Clang 3.4.

@Chilledheart

Yeah, it works fine with me (tested with both of my ubuntu 12.04 and osx 10.9.2 , clang 3.4 of course). cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.