<div dir="ltr"><div dir="ltr">On Wed, Apr 7, 2021 at 11:50 PM Piyush Aggarwal <<a href="mailto:piyushaggarwal002@gmail.com">piyushaggarwal002@gmail.com</a>> wrote:<br></div><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 dir="ltr"><div dir="ltr"><div style="font-family:arial,sans-serif"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 7, 2021 at 3:04 PM Piyush Aggarwal <<a href="mailto:piyushaggarwal002@gmail.com" target="_blank">piyushaggarwal002@gmail.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">I'll get right on it</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 7 Apr, 2021, 3:02 pm Ben Cooksley, <<a href="mailto:bcooksley@kde.org" target="_blank">bcooksley@kde.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 dir="ltr">Hi all,<div><br></div><div>Recently changes were made to kdeconnect-kde that introduced new dependencies that broke the build on Windows.</div></div></blockquote></div></blockquote><div><br></div><div style="font-family:arial,sans-serif">This should be fixed by the <a href="https://invent.kde.org/network/kdeconnect-kde/-/commit/80178b8ff80c1e42692b309880d6f1435c62ce4e" target="_blank">https://invent.kde.org/network/kdeconnect-kde/-/commit/80178b8ff80c1e42692b309880d6f1435c62ce4e</a> . Please poke the CI once so I can be sure.</div></div></div></blockquote><div><br></div><div>Thanks. I've taken a look and it appears that it has corrected the build failure - which means we are now in the same situation as earlier, where the testing phase cleans up the dbus-daemon processes that are launched by the CMake configure phase (which stops it from breaking the node entirely, but is still not perfect)</div><div> </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="ltr"><div class="gmail_quote"><div style="font-family:arial,sans-serif"><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 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 dir="ltr"><div><br></div><div>While this in itself is an issue, it has exposed a much more significant issue which requires urgent remediation as it breaks any Windows build node that attempts to carry out a build of kdeconnect. These breakages require manual Sysadmin intervention to correct.</div></div></blockquote></div></blockquote><div><span class="gmail_default" style="font-family:arial,sans-serif"><br></span></div><div><span class="gmail_default" style="font-family:arial,sans-serif">This issue needs  to be fixed appropriately. I wrote a way to kill dbus-daemon.exe with Win APIs in </span><span class="gmail_default" style="font-family:arial,sans-serif">the kdeconnect-kde source, used here: <a href="https://github.com/KDE/kdeconnect-kde/blob/master/indicator/main.cpp#L106" target="_blank">https://github.com/KDE/kdeconnect-kde/blob/master/indicator/main.cpp#L106</a> . The Function's definition is here: <a href="https://github.com/KDE/kdeconnect-kde/blob/master/indicator/indicatorhelper_win.cpp#L48" target="_blank">https://github.com/KDE/kdeconnect-kde/blob/master/indicator/indicatorhelper_win.cpp#L48</a> . Please let me know if I can be of help and integrate it somewhere in Craft/ the CI.<br></span></div></div></div></blockquote><div><br></div><div>The C++ code is completely innocent in this case, one of the checks being run by CMake as part of the configure-time step is the cause of this issue.</div><div>Do you have any kdeconnect-kde specific checks in there that call custom binaries / code to search for specific dependencies by any chance?</div><div> </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="ltr"><div class="gmail_quote"><div><span class="gmail_default" style="font-family:arial,sans-serif"></span></div><div><span class="gmail_default" style="font-family:arial,sans-serif"><br></span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><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 dir="ltr"><div>As this creates persistent issues for other projects and because a new dependency was introduced without notification, the following is notice that Windows CI support for kdeconnect-kde will be terminated in 12 hours time should steps to resolve this not be taken.</div></div></blockquote></div></blockquote><div><br></div><div style="font-family:arial,sans-serif">I apologize for the mishap and the inconvenience caused to the other projects. This was not intended.</div><div style="font-family:arial,sans-serif"><br></div></div><div class="gmail_quote"><div style="font-family:arial,sans-serif">Best</div><div style="font-family:arial,sans-serif">Piyush Aggarwal</div><div style="font-family:arial,sans-serif">from the KDE Connect Team<br></div><br></div></div></blockquote><div><br></div><div>Thanks,</div><div>Ben </div></div></div>