<div dir="ltr">Hey,<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 16, 2016 at 1:38 AM, Kohei Yoshida <span dir="ltr"><<a href="mailto:libreoffice@kohei.us" target="_blank">libreoffice@kohei.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, 2016-09-14 at 09:33 +0200, Eike Rathke wrote:<br>
> <br>
> > For anyone wondering the only two changes against 1.2.1 are:<br>
> ><br>
> > <a href="https://gitlab.com/mdds/mdds/commit/22c0057292431e76fef690ed2856c3a" rel="noreferrer" target="_blank">https://gitlab.com/mdds/mdds/<wbr>commit/<wbr>22c0057292431e76fef690ed2856c3<wbr>a</a><br>
> > cb8eac20e<br>
> > <a href="https://gitlab.com/mdds/mdds/commit/0d840a43d062f0df033c10e115a6d6e" rel="noreferrer" target="_blank">https://gitlab.com/mdds/mdds/<wbr>commit/<wbr>0d840a43d062f0df033c10e115a6d6<wbr>e</a><br>
> > 37de754da<br>
> ><br>
> > I suppose that there is no difference to a normal bug fix at least<br>
> > for 5.2<br>
> > as there are no new features in the code. If nobody complains I<br>
> > would<br>
> > backport the change to 5-2 soon.<br>
> Fine with me.<br>
<br>
</span>So, Markus backported the commit to the 5.2, the build of which<br>
initially failed because the 5.2 branch contained a patch that was<br>
already included in mdds 1.2.2 (one from Caolan).  I've removed that<br>
patch and it now builds fine on Linux and Windows.  But it fails on OSX<br>
due to reasons probably unrelated to the mdds change.<br>
<br>
<a href="http://ci.libreoffice.org/job/lo_gerrit_master/17046/" rel="noreferrer" target="_blank">http://ci.libreoffice.org/job/<wbr>lo_gerrit_master/17046/</a><br>
<br>
It's been a while since I handled something like this.  Could someone<br>
remind me what we typically do in such cases?  I would personally think<br>
that it's probably fine to just dismiss the build failure and go on<br>
ahead (given the scope of the change being so limited), but I'd like to<br>
double-check it and see if I'm sane. ;-)<br>
<span class="HOEnZb"><font color="#888888"><br>
<br></font></span></blockquote><div><br></div><div>The problem has been solved by a rebase and the patch has been pushed.<br><br></div><div>Markus <br></div></div><br></div></div>