<div dir="ltr">So I think I have resolved most of the issues I was finding, still (at least) one more however :-)<div><br><div>We seem to have developed a dependency on libzstd.dll I think this is a new dependency of qt from 5.13 onwards.</div><div>This wasn't picked up in the build, but when I run it is reported as an error.</div><div><br></div><div>Currently libzstd.dll is not generated in MXE, but it looks like someone filed a pull request that fixes that 5 days ago, but as of this morning it hasn't been merged. When that is available I will see if I can get something that actually executes.</div><div>Not sure if I will need to manually add the dll as an install file, or if it will automatically be picked up..</div><div><br></div><div>I contacted the MXE mailing list about getting cmake updated, and they were quite helpful, there is a mechanism in MXE for trying updated packages of things in MXE, so I will try building with an up to date CMake/ In the meantime my branch has the modified findpkgconfig.cmake added to the container build.<br></div><div>I will send a draft pull request in case anyone wants to make sure I am not going in completely the wrong direction.</div><div><br></div><div>Paul</div><div><br></div><div>I am going to file a draft pull request so you can </div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 28, 2020 at 6:53 PM Dirk Hohndel <<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>I'd start with b, then d<br><br>5.15 is something we'll need to get to eventually.<br>In the past there was an MXE contributor who tended to work on that, maybe reach out to them as well.<br><br><br>/D<br><br><br><div class="gmail_quote">On May 28, 2020 9:39:32 AM PDT, Paul Buxton <<a href="mailto:paulbuxton.mail@googlemail.com" target="_blank">paulbuxton.mail@googlemail.com</a>> wrote:<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr"><div dir="ltr">So down the rabbit hole!</div><div dir="ltr"><br><div>Taking a reasonably up to date version of MXE we hit a bug in Cmake 3.15 where when specifying a PKG_CONFIG_PATH with multiple folders in the path and cross compiling to windows (on a Linux host) it incorrectly swaps the : with a ; so pkg-config only searches the first entry on the path.</div><div>This issue has been fixed in CMake 3.17.</div><div><br></div><div>So should I,</div><div>a) Try getting cmake manually of a version we choose</div><div>b) Patch the cmake we have in the container (it's a 1 line fix)</div><div>c) Get MXE to use the newer cmake</div><div>d) Get MXE to cherry pick the fix</div><div>e) Something else</div><div><br></div><div>Options (c) and (d) would also end up pulling in QT 5.15, and when I briefly tried this yesterday it was failing to build qtbase...</div><div>Option(a) might have other unknowns.</div><div>Option (b) is a bit hacky :-)</div><div><br></div><div>Any preferences?</div><div><br></div><div>There is also an issue with missing symbol</div><div>libglib is missing _imp__timeGetTime</div><div>Which seems to be related to this change (or something similar) in the Mingw toolchain</div><div><a href="https://sourceforge.net/p/mingw-w64/mingw-w64/ci/65042f860f8cb67b26d97fd763143946f9a7c6e6/" target="_blank">https://sourceforge.net/p/mingw-w64/mingw-w64/ci/65042f860f8cb67b26d97fd763143946f9a7c6e6/</a><br></div><div><br></div><div>But I haven't figured out yet where mxe gets its mingw from, so haven't been able to oinpoint when/how this has been broken.</div><div><br></div><div><br></div><div><br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 28, 2020 at 6:59 AM Paul Buxton <<a href="mailto:paulbuxton.mail@googlemail.com" target="_blank">paulbuxton.mail@googlemail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Yeah, I saw the parameter and an using it locally, but as Dirk mentions it isn't used by the build on GitHub.<div dir="auto">I will include this in my changes when I get something that behaves.</div><div dir="auto"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 27 May 2020, 20:00 Dirk Hohndel, <<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Hi Salva,<div><br></div><div>you are of course correct. And I had used a script to pass that in here, but then never checked in that script.</div><div>As a result the GitHub Action happened to use 'master' - which wasn't really what I had intended.</div><div><br></div><div>/D<br><div><br><blockquote type="cite"><div>On May 27, 2020, at 10:52 AM, Salvador Cuñat <<a href="mailto:salvador.cunat@gmail.com" rel="noreferrer" target="_blank">salvador.cunat@gmail.com</a>> wrote:</div><br><div><div dir="auto"><div>Hi Dirk, Paul<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El mar., 26 may. 2020 17:30, Dirk Hohndel via subsurface <<a href="mailto:subsurface@subsurface-divelog.org" rel="noreferrer" target="_blank">subsurface@subsurface-divelog.org</a>> escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br></div><div>The MXE version is fixed as a specific SHA. And as I grep through the sources to refresh my memory where that's done I notice that the SHA doesn't appear to be in the source tree. How weird.</div><div>I believe the last one that I used (based on the remnants of local builds) was 9f6b9c6f - but I need to dig some more to figure out why this isn't in the sources...</div><div></div></div></blockquote></div></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">In aab658f we introduced a parameter for the docker build to pass the mxe git sha using --build-arg= </div><div dir="auto"><br></div><div dir="auto">After this change the used version is not in the Dockerfile any more, but it's manually entered while building the image.</div><div dir="auto"><br></div><div dir="auto">Best regards.</div><div dir="auto"><br></div><div dir="auto">Salva.</div></div>
</div></blockquote></div><br></div></div></blockquote></div>
</blockquote></div>
</blockquote></div><br>-- <br>From my phone</div></blockquote></div>