<div dir="ltr">David,<div><br></div><div>I did +1 to your MR, but you have to be the one merging it.</div><div><br></div><div>Yumin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 1, 2016 at 6:35 PM, David Thompson <span dir="ltr"><<a href="mailto:david.thompson@kitware.com" target="_blank">david.thompson@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> MR closed.<br>
<br>
Thanks, Rob.<br>
<div class="HOEnZb"><div class="h5"><br>
        David<br>
<br>
><br>
> On Wed, Jun 1, 2016 at 5:45 PM, David Thompson<br>
> <<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br>
>> Hi Rob,<br>
>><br>
>>> The shared_ptr test never compiling is orthogonal to the shiboken<br>
>>> issue so that is why I broke it out into a separate MR.<br>
>><br>
>> But we cannot merge both of them because MR 94 addresses both. It is a tiny fix and I would rather not spend more time on it. Do you really think it's important to rewrite MR 94?<br>
>><br>
>>        David<br>
>><br>
>>><br>
>>> On Wed, Jun 1, 2016 at 5:34 PM, David Thompson<br>
>>> <<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br>
>>>>> Here is the MR that fixes the issue ( that I created ):<br>
>>>>> <a href="https://gitlab.kitware.com/cmb/smtk/merge_requests/95" rel="noreferrer" target="_blank">https://gitlab.kitware.com/cmb/smtk/merge_requests/95</a><br>
>>>><br>
>>>> Umm... I submitted a similar MR here:<br>
>>>><br>
>>>> <a href="https://gitlab.kitware.com/cmb/smtk/merge_requests/94" rel="noreferrer" target="_blank">https://gitlab.kitware.com/cmb/smtk/merge_requests/94</a><br>
>>>><br>
>>>> that also fixes things in the event that boost shared pointers are used. The issue I don't want to deal with is figuring out how to avoid explicitly specifying the template parameter...<br>
>>>><br>
>>>>       Sorry,<br>
>>>>       David<br>
>>>><br>
>>>>><br>
>>>>> On Wed, Jun 1, 2016 at 5:04 PM, David Thompson<br>
>>>>> <<a href="mailto:david.thompson@kitware.com">david.thompson@kitware.com</a>> wrote:<br>
>>>>>>> Can you open an issue that provides / points me towards how the newer<br>
>>>>>>> clang compilers are failing the c++11 check for shared ptr?<br>
>>>>>>><br>
>>>>>>> That is something that should work going forward.<br>
>>>>>><br>
>>>>>><br>
>>>>>> Here's the error message:<br>
>>>>>><br>
>>>>>> Trying shared ptr; result: Change Dir: /Stage/Build/SMTK/polygon/CMakeTmp/CMakeFiles/CMakeTmp<br>
>>>>>><br>
>>>>>> Run Build Command:"/usr/local/bin/ninja" "cmTC_e0daa"<br>
>>>>>> [1/2] Building CXX object CMakeFiles/cmTC_e0daa.dir/shared_ptr.cxx.o<br>
>>>>>> FAILED: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++     -std=c++11  -arch x86_64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk -mmacosx-version-min=10.10 -o CMakeFiles/cmTC_e0daa.dir/shared_ptr.cxx.o -c /Stage/Source/SMTK/polygon/CMake/shared_ptr.cxx<br>
>>>>>> /Stage/Source/SMTK/polygon/CMake/shared_ptr.cxx:14:30: error: no matching function for call to 'make_shared'<br>
>>>>>> std::shared_ptr<float> f = std::make_shared(42.0f);<br>
>>>>>>                           ^~~~~~~~~~~~~~~~<br>
>>>>>> /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/memory:4747:1: note: candidate template ignored: couldn't infer template argument '_Tp'<br>
>>>>>> make_shared(_Args&& ...__args)<br>
>>>>>> ^<br>
>>>>>> 1 error generated.<br>
>>>>>> ninja: build stopped: subcommand failed.<br>
>>>>>><br>
>>>>>> If you really think it's worth an issue, feel free to open one but please don't assign it to me. :-)<br>
>>>>>><br>
>>>>>>      David<br>
>>>><br>
>><br>
<br>
</div></div></blockquote></div><br></div>