[ 
https://issues.apache.org/jira/browse/THRIFT-5135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17096737#comment-17096737
 ] 

Mario Emmenlauer commented on THRIFT-5135:
------------------------------------------

I think this PR is wrong, something does not make sense. The c++ version is 
already set to 11 unless you override it.

> Thrift compiler can't be compiled by CMAKE on OSX
> -------------------------------------------------
>
>                 Key: THRIFT-5135
>                 URL: https://issues.apache.org/jira/browse/THRIFT-5135
>             Project: Thrift
>          Issue Type: Bug
>          Components: Compiler (General)
>    Affects Versions: 0.14.0
>            Reporter: Jano Svitok
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Log:
> {code:java}
> cd /.../thrift/compiler/cpp
>  /.../thrift/compiler/cpp/src/thrift/generate/t_d_generator.cc:754:31: error: 
> non-aggregate type 'vector<std::__1::string>' (aka 'vector<basic_string<char, 
> char_traits<char>,
>  allocator<char> > >') cannot be initialized with an initializer list
>  vector<string> t_d_generator::d_reserved_words = {
>  ^ ~
>  1 error generated.
>  * 
>  ** BUILD FAILED **
> {code}
> [Stackoverflow|https://stackoverflow.com/questions/39022787/error-non-aggregate-type-vectorint-cannot-be-initialized-with-an-initialize]
>  [ 
> |https://stackoverflow.com/questions/39022787/error-non-aggregate-type-vectorint-cannot-be-initialized-with-an-initialize]suggest
>  to enable C++11.
> PR: https://github.com/apache/thrift/pull/2051 -- this enables C++11 only on 
> mac. Since the code indeed uses C++ (std::vector initializer list, see 
> https://en.cppreference.com/w/cpp/language/list_initialization), it might be 
> appropriate to enable it for all targets (remove the if() and endif() lines).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to