[cmake-developers] review request: fix-protobuf-threads

Matthew Woehlke matthew.woehlke at kitware.com
Fri May 24 15:49:22 EDT 2013


On 2013-05-21 15:54, Brad King wrote:
> On 05/08/2013 05:09 PM, Matthew Woehlke wrote:
>> After chatting with Marcus how to resolve ParaView link errors due to
>> things using Google Protobuf needing to link to pthread, I have updated
>> FindProtobuf.cmake to also find the pthread library on UNIX platforms
>>
>> and include it in PROTOBUF_LIBRARIES.
>> This should fix link errors in a number of projects and/or remove the
>> need for downstream users to add pthread themselves when using protobuf.
>>
>> The branch (name in summary) is on stage. Please have a look if this is
>> reasonable and can be merged to master once 2.8.11 is out.
>
> The change looks okay to me.  Please revise the commit message
> to start in "FindProtobuf:" and update the text accordingly.

Done.

> Then merge to 'next' for testing.

How do I do that? Based on the wiki, I unsuccessfully tried:

$ ssh git at cmake.org stage cmake merge -b next fix-protobuf-threads
not allowed: stage cmake merge -b next fix-protobuf-threads

-- 
Matthew




More information about the cmake-developers mailing list