<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2014-09-27 19:25 GMT+02:00 Michael Meeks <span dir="ltr"><<a href="mailto:michael.meeks@collabora.com" target="_blank">michael.meeks@collabora.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>Hi Tamas,<br>
<br>
On Sat, 2014-09-27 at 16:05 +0200, Zolnai Tamás wrote:<br>
</span><span>> It's the collada2gltf code which uses std::shared_ptr at many places<br>
> so it would be much work to replace all of them with<br>
> boost::shared_ptr, but actually can be done if necessary.<br>
<br>
</span><span>        Interesting; we should work out what is best to do there I guess.<br>
<br>
</span><span>>  I just thought packages are created with newer compilers, since they<br>
> generate more better output (in theory).<br>
<br>
</span><span>        We compile LibreOffice on our oldest Linux base-line, which is<br>
</span>something horribly old; that is because it is the only effective way to<br>
get backwards comaptibilioty<br>
<span><br>
> Other thing is that I need to know whether this is the problem indeed.<br>
> So can I know what compilers are used for packages? Are they support<br>
> std::shared_ptr?<br>
<br>
</span>        AFAICS it is bad form to disable the feature during the release series<br>
without discussing it and/or notifying the author etc. I imagine it was<br>
inadvertent, and/or Cloph who normally does the compiles is on vacation<br>
& Robinson has been standing in for a bit; I've no idea what happened<br>
here. I took a look at <a href="http://configure.ac" target="_blank">configure.ac</a> and distro-config's on the branch<br>
and couldn't see what happened there either.<br>
<br>
        What commit disabled that ?</blockquote><div><br></div><div>This is the commit which adds check for std::shared_ptr:<br><a href="http://cgit.freedesktop.org/libreoffice/core/commit/?id=aa5fe7958d087fbd6e64b29bbf2fa6e4d9ba5ab6" target="_blank">http://cgit.freedesktop.org/libreoffice/core/commit/?id=aa5fe7958d087fbd6e64b29bbf2fa6e4d9ba5ab6</a><br></div><div>Git log shows that I'm the author, but I actually did just one part of that patch. Matus finished it adding the std::shared_ptr check (next to other things). That's the point where we should have realized that collada2gltf uses C++11 and so should not be part of 4.3, but it seems in this case our division of labor lead to information loss.<br><br></div><div>Best Regards,<br></div><div>Tamás<br></div></div><br></div></div>