<div dir="auto"><div><div class="gmail_quote"><div dir="ltr">On Sat, 8 Sep 2018, 20:12 David Faure, <<a href="mailto:faure@kde.org">faure@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On vendredi 31 août 2018 10:50:53 CEST Ben Cooksley wrote:<br>
> On Tue, Aug 28, 2018 at 10:35 PM Ben Cooksley <<a href="mailto:bcooksley@kde.org" target="_blank" rel="noreferrer">bcooksley@kde.org</a>> wrote:<br>
> > Hi all,<br>
> <br>
> Hi all,<br>
> <br>
> > We currently have a severe bug in ECM's AddTest macro due to it's<br>
> > behaviour around the setting of QT_PLUGIN_PATH. On Windows it would<br>
> > appear that the code in ECMAddTest mangles the environment variable,<br>
> > removing crucial paths that the CI system includes within this<br>
> > variable which are needed for applications and tests to function.<br>
> > <br>
> > As a consequence of this, several projects tests are broken, and some<br>
> > of these only fail by timeout. This has the effect of blocking Windows<br>
> > CI builders for several hours in some instances simply due to tests<br>
> > that are hanging and timing out as a result of being unable to load<br>
> > plugins.<br>
> > <br>
> > Can we please either short circuit this logic (and not try to set<br>
> > QT_PLUGIN_PATH on Windows) or figure out the correct logic to be used<br>
> > in this instance?<br>
> > <br>
> > My suspicion is that CMake treats ';' specially, which means that when<br>
> > the macro re-includes the existing content of the variable it gets<br>
> > mangled by CMake, resulting in the entire content of that variable<br>
> > being rendered unreadable.<br>
> > <br>
> > Alternatively, CMAKE_LIBRARY_OUTPUT_DIRECTORY could have a trailing<br>
> > backslash, which has certainly caused issues elsewhere.<br>
> <br>
> Based on the lack of response to this, along with my earlier mail to<br>
> k-f-d 7 days ago it doesn't look like anyone's going to pick this up.<br>
> <br>
> As this issue is causing test breakages on the CI system including<br>
> tying up valuable builder time for extraordinarily excessive amounts<br>
> of time, i've taken a sledgehammer to the offending code in ECM.<br>
<br>
Is there any indication that this code was misbehaving on Unix as well?<br>
I'm surprised that you took out everything, not just the Windows case.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">As far as I've seen it hasn't caused any issues on Unix so it should be fine to restore there.</div><div dir="auto"><br></div><div dir="auto">I disabled it for all as a precautionary measure as I wasn't sure if it was causing issues on Unix systems as well.</div><div dir="auto"><br></div><div dir="auto">Cheers,</div><div dir="auto">Ben</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- <br>
David Faure, <a href="mailto:faure@kde.org" target="_blank" rel="noreferrer">faure@kde.org</a>, <a href="http://www.davidfaure.fr" rel="noreferrer noreferrer" target="_blank">http://www.davidfaure.fr</a><br>
Working on KDE Frameworks 5<br>
<br>
<br>
<br>
</blockquote></div></div></div>