<div dir="ltr"><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Andrew Caudwell</b> <span dir="ltr"><<a href="mailto:acaudwell@gmail.com">acaudwell@gmail.com</a>></span><br>Date: Thu, Mar 29, 2018 at 11:55 AM<br>Subject: Re: Bug#892861: libglm-dev: removal of default type initialization breaking packages<br>To: Guus Sliepen <<a href="mailto:guus@debian.org">guus@debian.org</a>><br><br><br><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Thu, Mar 29, 2018 at 9:12 AM, Guus Sliepen <span dir="ltr"><<a href="mailto:guus@debian.org" target="_blank">guus@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tags 892861 - wontfix<br>
thanks<br>
<span><br>
On Wed, Mar 28, 2018 at 01:19:44PM +1300, Andrew Caudwell wrote:<br>
<br>
> Upstream has implemented my suggestion to re-add default initialization as<br>
> opt-in via a new define:<br>
><br>
> <a href="https://github.com/g-truc/glm/issues/735" rel="noreferrer" target="_blank">https://github.com/g-truc/glm/<wbr>issues/735</a><br>
> <a href="https://github.com/g-truc/glm/commit/8390a77b3a278b15259e5ca6e67f7e41badc457b" rel="noreferrer" target="_blank">https://github.com/g-truc/glm/<wbr>commit/8390a77b3a278b15259e5ca<wbr>6e67f7e41badc457b</a><br>
><br>
> Could you apply the commit as a patch so maintainers can then define<br>
> GLM_FORCE_CTOR_INIT and avoid having to modifying code?<br>
<br>
</span>Sure. However, I'm running into problems trying to apply the patch: it<br>
doesn't apply cleanly on 0.9.9~a2, and if I just package the latest<br>
revision from git, then I am getting internal compiler errors from GCC.<br>
I can still compile 0.9.9~a2 without problems, so I will try to see if I<br>
can just adapt the commit which adds GLM_FORCE_CTOR_INIT to 0.9.9~a2.<br>
<span><br></span></blockquote></span><div><div>Here's a patch I created. The main issue is they added a few macros before the constructors that we need to remove:<br><br>git format-patch -1 8390a77b3a278b15259e5ca6e67f7e<wbr>41badc457b<br>perl
 -npe "s/GLM_CONSTEXPR_(CTOR_)?CXX14 //g" 
0001-Added-GLM_FORCE_CTOR_<wbr>INIT-735-740.patch > 
0001-Added-GLM_FORCE_CTOR_<wbr>INIT-735-740-fixed.patch<br><br>I also had to remove some expected white space from type_mat3x2.inl and type_mat4x2.inl.<br><br></div><div>Tested it against the 0.9.9-a2 tag with GLM_FORCE_CTOR_INIT defined and it fixes my issue.</div><br> </div><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
> Let me know as then I can then avoid having to embed the current release in<br>
> my software.<br>
<br>
</span>Yeah, that would be less than optimal.<br>
<div class="m_-3232580369874301191HOEnZb"><div class="m_-3232580369874301191h5"><br>
--<br>
Met vriendelijke groet / with kind regards,<br>
      Guus Sliepen <<a href="mailto:guus@debian.org" target="_blank">guus@debian.org</a>><br>
</div></div></blockquote></span></div><br></div></div>
</div><br></div>